Conversation
There was a problem hiding this comment.
현재 .env 파일이 저장소에 포함되어 API 키가 노출되어있습니다! 보안상 위험하기 때문에, 기존 키는 폐기하시고 새로 발급받아 사용해주세요 🙂 또한 재발급 하시고 .env 파일이 GitHub에 업로드되지 않도록 .gitignore에 .env를 추가해주시면 이 파일은 업로드되지 않을 겁니다 !
| <LoadingSpinner /> | ||
| </div> | ||
| )} | ||
| {!isPending && ( |
There was a problem hiding this comment.
이렇게 하셔도 기능적으로는 문제가 없지만 구조가 조금 중복되어 가독성 떨어질 수 있습니다 !
{isPending && <div className='flex justify-center items-center h-dvh'><LoadingSpinner /></div>}
<div className='p-10 grid gap-4 grid-cols-2 sm:grid-cols-3 ...'>
{movies.map((movie) => (
<MovieCard key={movie.id} movie={movie} />
))}
</div>
이렇게 정리하시면 더 깔끔한 코드를 사용할 수 있습니다 !
There was a problem hiding this comment.
Vite로 생성 시 기본적으로 제공되는 스타일은 사용되지 않으니 제거해 주시는 것이 좋습니다 !
| const MovieDetailPage = () => { | ||
| const params = useParams(); | ||
|
|
||
| console.log(params); |
There was a problem hiding this comment.
console문은 pr 업로드 시 제거해주시는 것이 좋습니다
There was a problem hiding this comment.
현재 HomePage는 실제 메인 페이지라기보다는 Navbar와 Outlet을 포함한 레이아웃 역할을 하고 있어, HomePage보다는 Layout/ 폴더를 생성해서 Layout과 같은 이름으로 변경해서 관리하는 것이 더 적절해 보입니다 !
yewon20804
left a comment
yewon20804
left a comment
There was a problem hiding this comment.
미션 1, 2 모두 수고 많으셨습니다! 😊
현재 PR에 두 미션 내용이 함께 포함되어 있는데, 다음 주차부터는 미션 1과 미션 2를 각각 분리하여 PR을 올려주시면 좋을 것 같습니다 !
또한 API 키가 노출된 부분이 있어 해당 코멘트 확인 후 꼭 수정 부탁드립니다 !
미션 하시면서 페이지네이션과 카테고리별 동적 이동 기능도 잘 구현해주셨습니다 👍
코멘트로 안내드린 부분만 보완해주시면 더욱 완성도 높은 코드가 될 것 같습니다 !
2 participants
📚 주차 / 미션
📌 작업 내용
✨ 상세 작업 내용
✅ 체크리스트