-
Notifications
You must be signed in to change notification settings - Fork 78
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
[3팀 박수연] [Chapter 1-1] 프레임워크 없이 SPA 만들기 #40
base: main
Are you sure you want to change the base?
Conversation
안녕하세요 저는 클래스를 사용해본적이 거의 없는데 덕분에 잘 구경할 수 있었습니다! navigateTo 함수의 pushState 매개변수 역할이 궁금했습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description에 적어주신 것처럼 책임을 분리해서 깔끔하게 구조를 잡으시려고 한게 느껴졌습니다!
특히 저는 하나의 파일에서 모든 페이지에 대한 render를 하는 로직이 있었는데, 수연님은 각 페이지 안에 render 하는 함수가 따로 있고, index.js에서 이를 export해주고 있다는 부분에서 그렇게 생각했어요!
과제하느라 고생하셨습니다!! 저도 많이 배우고 갑니다!ㅎㅎ
* refactor: 레이아웃 개선 * refactor: 이벤트 위임 개선 * fix: navigation 로그인 상태 반영
과제 체크포인트
기본과제
1) 라우팅 구현:
2) 사용자 관리 기능:
3) 프로필 페이지 구현:
4) 컴포넌트 기반 구조 설계:
5) 상태 관리 초기 구현:
6) 이벤트 처리 및 DOM 조작:
7) 라우팅 예외 처리:
심화과제
1) 해시 라우터 구현
2) 라우트 가드 구현
3) 이벤트 위임
과제 셀프회고
기술적 성장
코드 품질
학습 효과 분석
과제 피드백
리뷰 받고 싶은 내용
부족함이 많은 과제입니다 ㅠㅠ 내 직장 후임이다 생각하시고 편한 마음으로.. 날카롭게 지적해 주세요...!!!!!
이번 한 주도 고생 많으셨습니다! 감사합니다 🙇♀️ ❤️