Conversation
hyesngy
left a comment
hyesngy
left a comment
There was a problem hiding this comment.
10주차 워크북까지 정말 긴 여정이었습니다.
그동안 모두 고생 많으셨습니다! 👏🏻👏🏻👏🏻 짧지 않은 기간동안 의미 있는 시간이 되었기를 바랍니다.
이제 기본 워크북은 모두 완주하셨지만, 11-12주차 워크북도 도전해보시길 추천드립니다! 11주차에서는 Vercel을 활용한 배포와 CI/CD 구성을 다루고, 12주차에서는 WebSocket을 이용한 실시간 통신과 Cypress를 활용한 E2E 테스트를 다룹니다.
앞으로도 꾸준히 학습하고 성장하시길 응원하며, 데모데이까지 화이팅 입니다! 👍🏻👍🏻👍🏻
| import MovieDetailBackdrop from "./MovieDetailBackdrop"; | ||
| import MovieDetailInfo from "./MovieDetailInfo"; | ||
| import MovieDetailOverview from "./MovieDetailOverview"; | ||
| import MovieDetailCredits from "./MovieDetailCredits"; | ||
| import MovieDetailActions from "./MovieDetailActions"; |
There was a problem hiding this comment.
현재 components 폴더 내에서 Movie, Filter 등으로 도메인별 분리가 잘 되어 있고, Detail 폴더로 모달 관련 컴포넌트들을 세분화한 점이 좋습니다. 👍🏻
개인적으로 아쉬운 점은, 현재 모달 관련 컴포넌트들을 개별적으로 import하는 부분이 다소 깔끔하지 않게 보입니다. Detail 폴더 내에 index.ts 파일을 만들어서 관련 컴포넌트들을 한 번에 export하면, import 구문이 훨씬 깔끔해지고 폴더 구조도 더 체계적으로 관리할 수 있을 것 같아요!
| >로그아웃</button> | ||
| </div>; | ||
| }; | ||
|
|
There was a problem hiding this comment.
현재 코드를 보면 axiosInstance나 MyPage 파일이 있지만 사용되지 않고 있습니다. 현재 전반적으로 10주차 워크북 미션인 검색 기능에 집중한 구조로 되어 있는데, 지난 미션이었던 영화 페이지와 통합하면 더 완성도 높은 영화 사이트가 될 것 같습니다!
There was a problem hiding this comment.
🚀Challenge 미션에 따라 프로젝트 전체의 성능 최적화나 UX 개선에도 도전해보세요! 코드 품질 측면에서도 중복 제거, 책임 분리, 예외 처리 개선 등을 통해 가독성과 유지보수성이 좋은 코드로 리팩토링 해보면 어떨까요? 😊
2 participants
📝 미션 번호
10주차 Misson 1
📋 구현 사항
📎 스크린샷
10-1.mov
✅ 체크리스트
🤔 질문 사항