Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[김미진]Week5 #213

Open
wants to merge 8 commits into
base: part1-김미진
Choose a base branch
from

Conversation

mijin0928
Copy link
Collaborator

@mijin0928 mijin0928 commented Nov 19, 2023

요구사항

기본

  • [기본]https://bootcamp-api.codeit.kr/docs 에 명세된 “/api/sign-in”으로 { “email”: “[email protected]”, “password”: “sprint101” } POST 요청해서 성공 응답을 받고 “/folder”로 이동하나요?
  • [기본]이메일 input에서 “[email protected]”으로 “/api/check-email” 이메일 중복 확인 POST 요청하면 이메일 중복 에러를 확인할 수 있나요?
  • [기본]유효한 회원가입 형식의 경우 “/api/sign-up” POST 요청하고 성공 응답을 받으면 “/folder”로 이동하나요?

심화

  • [심화]로그인/회원가입시 성공 응답으로 받은 accessToken을 로컬 스토리지에 저장했나요?
  • [심화]로그인/회원가입 페이지에 접근시 로컬 스토리지에 accessToken이 있는 경우 “/folder” 페이지로 이동하나요?

주요 변경사항

  • 4주차 위클리미션 수정사항 반영하여 코드 리팩토링
  • 로그인, 회원가입 api

멘토에게

  • 요구사항 전부 구현해보려고 했는데 중복체크 구현하는 부분에서 409오류가 계속 나서 실패해서 결국 하지 못했습니다 어떻게 해야할까요?
  • 나머지 요구사항도 하기는 했는데 너무 어려워서 올바르게 구현된건지 잘모르겠습니다ㅠㅠ
  • 5주동안 멘토링 진행하면서 잘 몰랐던 부분도 알게 되었고 많이 배울 수 있어서 좋았습니다 그동안 감사했습니다!

mijin0928 and others added 7 commits November 5, 2023 16:59
- label태그의 for값과 input태그의 id값 수정
- form태그안에 div태그의 클래스명 수정
- 파일경로 수정
 refactor(mentor):
 - / 제거했습니다
refactor(mentor)
- 모듈 적용
- 수정사항 반영하여 코드 리팩토링
@mijin0928 mijin0928 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Nov 19, 2023
@mijin0928 mijin0928 changed the title [김미진]week5 [김미진]Week5 Nov 19, 2023
@mijin0928 mijin0928 force-pushed the part1-김미진-week5 branch from d5835f2 to 0591d6e Compare November 19, 2023 18:25
@mijin0928 mijin0928 force-pushed the part1-김미진-week5 branch from 0591d6e to 7d83e0f Compare November 20, 2023 02:59
// 눈 아이콘 클릭 시 이미지 및 타입 변경
function eyeToggle(e){
e.preventDefault();

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indexOf도 틀린건 아니지만 맥락상으로는 includes가 더 맞겠네요. 아래처럼 별도의 분기처리(if, else)없이 리팩토링 가능합니다.

const eyeIconSrc = e.target.src.includes('off') ? "./images/ico-eye-on.svg" : "./images/ico-eye-off.svg";
const passwordInputType = e.target.src.includes('off') ? "password" : "text";
e.target.src = eyeIconSrc;
pwInput.type = passwordInputType;    


if(localStorage.getItem('accessToken')) join(email, password);
}catch(error){
console.log(error);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

단순히 console을 찍을게 아닌, 사용자에게 에러가 났으니 다시 어떤 행동을 하도록 유도해주셔합니다. (ex. 새로고침)

@@ -0,0 +1,65 @@
const pwCheckInput = document.getElementById('signup-check-password');
const correct = {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

patternuserInfo는 다른 성질의 데이터입니다. 분리해주세요.

@lunaticscode
Copy link
Collaborator

  • 요구사항 전부 구현해보려고 했는데 중복체크 구현하는 부분에서 409오류가 계속 나서 실패해서 결국 하지 못했습니다 어떻게 해야할까요?

: 409오류는 Status 의미로 "Confilct" 입니다. 일단은 HttpStatusCode에 대한 이해가 필요하고, 중복된 데이터를 보내서 충돌이 일어났을 때 해당에러를 내뱉는 경우가 맞다면, 오류라고 볼 수 없고 서버에서 의도한 에러라고 판단해야됩니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants