[임주민] week5#247
Open
imkrmin wants to merge 11 commits intocodeit-bootcamp-frontend:mainfrom
Hidden character warning
The head ref may contain hidden characters: "part2-\uc784\uc8fc\ubbfc-week6"
Open
Conversation
imkrmin
requested review from
00eun and
domuk-k
and removed request for
00eun
| } | ||
| localStorage.setItem("accessToken", accessToken); | ||
| location.href = "/folder"; | ||
| } catch { |
Collaborator
There was a problem hiding this comment.
해당 catch 블록 내부에서는 서버 측 에러도 포함됩니다. 즉 사용자에게 이메일을 확인해주세요, 비밀번호를 확인해주세요 라는 메시지를 무조건적으로 던져준다는 것은 전적으로 사용자에게 잘못이 있다는 뜻이 되므로 5xx 상태코드에 대해서도 예외처리가 필요합니다.
| ); | ||
|
|
||
| function validatePasswordInput(password) { | ||
| if (password === "" || !isPasswordValid(password)) { |
Collaborator
There was a problem hiding this comment.
password 가 빈 문자열인 것 조차도 isPasswordValid 함수 내부에서 체크할 수 있도록 만들어주시면 코드가 더 간략해집니다.
| const signForm = document.querySelector("#form"); | ||
| signForm.addEventListener("submit", submitForm); | ||
| async function submitForm(event) { | ||
| event.preventDefault(); |
Collaborator
There was a problem hiding this comment.
다음부터는 굳이 form 태그를 사용하지 않으면 event.preventDefault() 코드를 쓸 필요 없어지니 참고해주세요.
| @@ -0,0 +1,52 @@ | |||
| const SIGN_INPUT_ERROR_CLASSNAME = "sign-input-error"; | |||
Collaborator
There was a problem hiding this comment.
SIGN_INPUT_ERROR_CLASSNAME , ERROR_MESSAGE_CLASSNAME, EMAIL_REGEX 는 상수 개념이므로 src/const.js에서 관리해주시면 되겠습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요구사항
기본
네비게이션 바, 푸터가 랜딩 페이지와 동일한 스타일과 규칙으로 만들어졌나요? (week 1 ~ 3 요구사항 참고)
상단 네비게이션 바에 프로필 영역의 데이터는 https://bootcamp-api.codeit.kr/docs 에 명세된 “/api/sample/user”를 활용했나요?
상단 네비게이션 바에 프로필 영역의 데이터가 없는 경우 “로그인” 버튼이 보이도록 만들었나요?
폴더 소유자, 폴더 이름 영역, 링크들에 대한 데이터는 “/api/sample/folder”를 활용했나요?
Static, no image, Hover 상태 디자인을 보여주는 카드 컴포넌트를 만들었나요?
Hover 상태에서 이미지가 기본 크기의 130%로 커지나요?
카드 컴포넌트를 클릭하면 해당 링크로 새로운 창을 띄워서 이동하나요?
Tablet에서 카드 리스트가 화면의 너비 1124px를 기준으로 이상일 때는 3열로 작을 때는 2열로 배치되나요?
Tablet, Mobile에서 좌우 최소 여백은 32px 인가요?
심화
주요 변경사항
스크린샷
멘토에게