[8주차] Team IPX 과제 제출합니다.#10
Open
GirimNam wants to merge 136 commits into
Open
Conversation
init: 프로젝트 초기 설정
Feat/login
Feat/signup
[FEAT] 회원가입 api 연동
[FEAT] 비로그인 차단 및 에러 모달창 구현
[Feat] 회원가입 필드별 에러 메시지 및 완료 모달 추가
[FEAT] 투표 페이지 api 연동 완료
[FEAT] 결과 페이지 반응형 레이아웃 적용
[FEAT] ABOUT US 페이지 완료
fix: login 페이지 suspense 추가
[DEV] dev 브랜치 최종 수합
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
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.
🔗배포링크
🎨참고 피그마
💻api 명세서
🗒️발표자료
이번주 소감
기림
voting을 클릭한다면, 투표를 하는 페이지(voting/leader나voting/demoday)가 아니라 결과 페이지(voting/result)로 이동하도록 구현하였고, 이를 위해 localstorage를 사용했습니다. 하지만 다른 기기에서 로그인해voting를 클릭한다면 다시 투표를 하는 페이지(voting/leader나voting/demoday)로 이동한다는 문제가 발생했습니다. 하지만 이를 너무 늦게 알게되어 api 관련 수정을 하기보다는 이런 상황이 발생한다면 에러 메시지를 화면에 띄우지 않고voting/result로 바로 push하도록 처리하였습니다. 이를 통해 localhost에서만 반복적으로 test하는 것이 아니라 배포 환경을 적극적으로 이용해야함을 다시 한번 깨달았고 실제 프로젝트에서는 api 명세서를 읽고 발생 가능한 경우를 최대한 생각하며 이와 관련하여 새롭게 규칙을 만들어야겠다고 생각했습니다.민서