-
Notifications
You must be signed in to change notification settings - Fork 8
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
[5, 6주차/다일] 워크북 제출합니다. #65
Conversation
chapter04 미션하면서 개인적으로 추가했던 무한스크롤 기능에 throttling으로 렌더링 최적화 를 시도했습니다. 아직 추가 수정이 필요합니다.
무한 스크롤 기능을 react-intersection-observer 패키지를 사용하여 구현하도록 수정했습니다.
form에서 사용할 요소들을 만들었습니다 useform방식 외에도 하드코딩으로 먼저 구현했습니다.
5주차 요구사항은 다 맞췄고, api설정을 리팩토링하면서 signupPage를 완성해서 테스트 했습니다
chapter06의 미션 초기 프로젝트 설정
쿠키를 사용해 token들을 저장하고 refresh token은 httpOnly 옵션을 주어서, 보안을 높이려 하였으나 백엔드에서 지금은 httpOnly의 내용을 받아줄 수 없기 때문에 구현만 하고 주석처리해서 accessToken과 같은 방식으로 refreshToken을 저장하고 사용하고 있습니다. 백엔드만 수정하면 적용되도록 해두었습니다.
검색기능을 구현하면서 기존 MovlieList 컴포넌트를 Widget레이어로 이동 시켰고, 기존에 적용되어있던 무한스크롤 로직을 수정했습니다. MovieList는 초기에 검색은 고려되지 않았기 때문에 useMovieList api함수의 로직이 수정되었습니다.
debounce를 추가하긴 했는데, 제가 구현한 로직상 검색에 debounce가 맞지 않기때문에 적용은 하지 않았고 쿠키가 똑바로 삭제되지 않는 문제를 수정하고, 일부 스타일 수정했습니다
MovieList 데이터를 받아오면서 loading state가 잘못 관리되고 있어 skeleton ui가 똑바로 출력되지 않던 이슈를 해결했습니다
일부 코드 정리 했습니다. 기능은 변한 부분 없습니다
app 의 style파일을 정리하면서 appcontainer를 추가하여 sticky 속성이 똑바로 작동하도록 함
recoil을 적용해봤습니다
todoList 리팩토링은 recoil로 했습니다. |
todoList에서 state를 얕은복사하여 잘못 사용하던 부분 이있어 수정했습니다.
05 06 chapter 키워드 추가 했습니다
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.
다일 수고하셨습니다! debounce, throttle을 커스텀 훅으로 구현 잘 해주셨네요 ㅎㅎ 워낙 잘하시니까... 별로 드릴 말씀은 없구 폴더 구조가 신기한데 혹시 어떻게 구성하셨는지 댓글 한 번만 달아주실 수 있나요??
기본적으로 FSD 디자인패턴을 적용한 것인데,
여기까지가 제가 FSD를 공부하면서 기록했던 내용들이고
그리고 FSD는 이런 Layer, Slice 구조를 위해 기본적으로 하나의 요소는 1개의 폴더로서 존재하고 내부에 segment가 존재하여 |
FSD패턴 자체가, 디자인 패턴 치고 러닝커브가 좀 있는 편이지만, |
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트
📌 주안점