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

[4주차/다일] 워크북 제출합니다 #55

Merged
merged 8 commits into from
Nov 4, 2024

Conversation

RyuDongHo
Copy link

@RyuDongHo RyuDongHo commented Nov 1, 2024

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

3주차 미션에서, 기능별로 파일을 분할했으나 MovieListPage컴포넌트에서
api로서 작동하도록한 부분의 기능이 예쁘게 분리되지 않았던 점을 수정하고
4주차 미션에서 작업하게 될 MovieDetailPage컴포넌트의 초기 파일을 추가해서
3주차 미션 파일을 복사해왔습니다
chapter03과 chapter04가 서로 복사된 내용이라
실수로 수정이 겹쳐있습니다
movie detail정보를 가져오는 api를 추가했습니다.
MovieDetailPage컴포넌트를 추가하면서 api를 리팩토링했습니다
추가로 MovieListPage의 무한 스크롤을 구현했지만
새로운 영화 데이터를 가져올 때 기존의 MovieList가 처으부터 렌더링되는
이슈가 있습니다.
추후 수정예정입니다.
movieList의 무한스크롤 기능 완료후 적용했습니다
globalStyle에서 들어가있는 css reset이 h1~h3 태그도 전부 스타일을 지워버리는 점을
수정해서 설정을 되돌렸습니다
MovieDetailPage 컴포넌트를 완성했습니다
현재 프로젝트에 적용된 FSD 디자인 패턴을
더 보기쉽게 폴더명을 수정했습니다
Copy link

@lsy1307 lsy1307 left a comment

Choose a reason for hiding this comment

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

군더더기 없이 깔끔한 코드네요! 수고하셨습니다!

@RyuDongHo RyuDongHo merged commit 581ac3b into UMC-Inha:dail/main Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chapter04_useNavigate, useParams, useLocation, custom-hook 사용 방법
2 participants