Feat: 3주차 미션 1 - #23
Conversation
1주차 미션1 - 뮤
Feat: 2주차 미션1 - 뮤
Feat: 2주차 미션2 - 뮤
| 'https://api.themoviedb.org/3/movie/popular?language=ko-KR&page=1', | ||
| { | ||
| headers: { | ||
| Authorization: `Bearer eyJhbGciOiJIUzI1NiJ9.eyJhdWQiOiJjYzVjY2QxZjMyMDVlZmFjODkyYzVlNTgxNWY1MjY4OSIsIm5iZiI6MTc3NDk5Njk3OS43MTEsInN1YiI6IjY5Y2M0ZGYzYTI1M2YwYzcyNGVkNDdlOSIsInNjb3BlcyI6WyJhcGlfcmVhZCJdLCJ2ZXJzaW9uIjoxfQ.APchjMzIRObS5T04QcUSNIveb8Kq3D0QGhwvpeJM0y0`, |
There was a problem hiding this comment.
현재 API 토큰 값을 하드코딩으로 넣으셨는데, 이 상태로 깃허브에 올라가면 키 탈취 위험이 있어 위험합니다!
이러한 키 값은 .env 파일을 생성하여 VITE_TMDB_KEY=키값 으로 키 값을 정의하고, Authorization에서 변수 값을 import하는 방식이 적절합니다! 참고로 .env 파일은 .gitignore에 추가해서 깃허브에 업로드 안되도록 처리해주셔야 합니다! 또한 현재 키 값은 이미 노출이 되었으니 새로 재발급 받고 진행해주시면 됩니다!
| const MoviesPage = () => { | ||
| const [movies, setMovies] = useState<Movie[]>([]); | ||
|
|
||
| console.log(movies); // 영화 데이터 체크 |
There was a problem hiding this comment.
console문은 pr 업로드 시 제거해주시는 것이 좋습니다
There was a problem hiding this comment.
각 페이지마다 tailwind로 스타일을 작성해주셨기 때문에 사용하지 않는 스타일은 제거해주시는게 좋아요 !
| @@ -0,0 +1,33 @@ | |||
| import './App.css'; | |||
There was a problem hiding this comment.
여기도 App.css 스타일을 사용하지 않으니 제거해주시는 것이 좋을 것 같습니다
There was a problem hiding this comment.
여기도 Vite가 기본으로 생성해주는 스타일이 그대로 남아 있는데 사하지 않는 스타일은 지워주시는게 좋습니다 !
yewon20804
left a comment
yewon20804
left a comment
There was a problem hiding this comment.
미션 1 수고하셨습니다! 😊
components, layout, pages, types로 폴더 구조를 잘 나누어 구성해주신 점이 좋았습니다 !
코멘트에도 남겼지만, API 키가 코드에 노출된 부분은 보안상 위험해서 꼭 확인하시고 수정 부탁드립니다 !
Feat: Week3/nonshaman m2
Revert "Feat: Week3/nonshaman m2"
feat: 1주차 미션
Feat: 2주차 미션1
Week2/sori m2
Week3/sori m3
3 participants
📚 주차 / 미션
📌 작업 내용
✨ 상세 작업 내용
📸 스크린샷
❓ 리뷰어가 알아야 할 사항 / 질문
✅ 체크리스트