Skip to content

[3주차/지니] 워크북 제출합니다. - #34

Merged
wldnjs12 merged 4 commits into
UMC-Inha:지니/mainfrom
wldnjs12:submit2
Oct 7, 2025
Merged

wldnjs12 merged 4 commits into
UMC-Inha:지니/mainfrom
wldnjs12:submit2

Conversation

@wldnjs12

@wldnjs12 wldnjs12 commented Oct 2, 2025 •

Copy link
Copy Markdown

[

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

export default function MoviesPage() {
const [movies, setMovies] = useState<Movie[]>([]);
const [loading, setLoading] = useState(true);
const [err, setErr] = useState<string | null>(null);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null 처리를 통해서 404와 콘솔에러 를 차단하신점이 인상적입니다!!

@@ -0,0 +1 @@
VITE_TMDB_TOKEN=eyJhbGciOiJIUzI1NiJ9.eyJhdWQiOiJlMDMzYmE1ZWE4NWJhN2E2Nzc1ODI1ZGY4OGJhNDkwMCIsIm5iZiI6MTc1OTM4Nzc2Mi4wNTksInN1YiI6IjY4ZGUyMDcyNTZlODIxMzkwZmZkMjEwNSIsInNjb3BlcyI6WyJhcGlfcmVhZCJdLCJ2ZXJzaW9uIjoxfQ.JVCyPuBdfRl0tIePXpbj1xyTGppUPQaOiv6vSM0yUyQ No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

API 키와 같은 민감한 정보는 보안상 소스코드에 직접 노출되면 위험합니다.
그래서 .env 파일은 절대 GitHub에 공개되면 안 되므로, chapter03/mission1/.gitignore에 .env를 꼭 추가해 gihub에는 올라오지 않고 로컬에서만 사용할 수 있도록 해주세요!

Comment on lines +21 to +28
const title = useMemo(() => {
switch (kind) {
case "popular": return "인기 영화";
case "now_playing": return "상영 중";
case "top_rated": return "평점 높은";
case "upcoming": return "개봉 예정";
}
}, [kind]);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

useMemo로 타이틀 매핑한 부분이 좋네요 !!

@sunnyinha

Copy link
Copy Markdown
Collaborator

지니, 3주차도 수고하셨습니다!
현재 .env가 이미 GitHub에 올라간 상태라, .gitignore에 추가만 해서는 해결되지 않습니다.
github에 올라간 .env 파일은 직접 제거해주세요!
해당 부분 수정 후에 머지해주세요~

@wldnjs12
wldnjs12 merged commit aa97597 into UMC-Inha:지니/main Oct 7, 2025
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.

Chapter03_부드럽고 강력한 웹앱 만들기: Tailwind CSS, React Router, API 통신

4 participants