Skip to content

Conversation

@dasosann
Copy link
Contributor

요약

구현 사항

  • 코드리뷰 반영
  • [ ]
  • [ ]
  • [ ]

📸 스크린샷

Need Review

  • ~ 부분 이렇게 구현했어요, 피드백 부탁해요!

Reference

📜 리뷰 규칙

Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.

  • P1: 꼭 반영해주세요 (Comment)
  • P2: 적극적으로 고려해주세요 (Comment)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@vercel
Copy link

vercel bot commented Jan 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
fe Ready Ready Preview, Comment Jan 21, 2026 7:36am

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @dasosann, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 주로 코드 리뷰 피드백을 반영하여 코드 품질을 개선하고 불필요한 코드를 제거하는 데 중점을 둡니다. 앨범 생성 시 날짜 처리 로직을 date-fns 라이브러리를 활용하여 현대화하고, 사진 상세 보기 화면에서 디버깅용 코드를 정리하며, 사진 삭제 후의 UI 상태 관리 로직을 최적화했습니다. 전반적으로 코드의 가독성과 유지보수성을 향상시키는 변경 사항들입니다.

Highlights

  • 날짜 포맷팅 개선: 앨범 생성 입력 목록에서 수동으로 구현했던 날짜 포맷팅 로직을 date-fns 라이브러리를 사용하여 더 간결하고 안정적으로 변경했습니다.
  • 디버그 코드 제거: 사진 상세 정보 푸터 컴포넌트에서 디버깅 목적으로 사용되던 console.log 문을 제거했습니다.
  • 사진 삭제 후 상태 관리 간소화: 사진 삭제 성공 후 setIsPhotoInfoOpen(false) 호출을 제거하여 사진 정보 패널의 상태 관리 로직을 간소화했습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

코드 리뷰 내용을 잘 반영해주셨습니다. date-fns를 사용한 날짜 형식 리팩토링과 불필요한 console.log 및 상태 업데이트 로직 제거로 코드가 더 깔끔해졌습니다.

추가적으로 사진 삭제 로직을 검토하던 중, 데이터 정합성 관련하여 개선할 점을 발견하여 코멘트를 남겼습니다. handleDelete 함수에서 관련된 모든 react-query 캐시를 무효화하여 UI가 최신 상태를 올바르게 반영하도록 하는 것이 중요합니다. 확인 부탁드립니다.

@@ -62,7 +55,6 @@ export default function FooterPhotoDetail({
try {
await mutateAsyncDelete({ albumId, photoId });
queryClient.invalidateQueries({ queryKey: [EP.album.photos(albumId)] });
Copy link
Contributor

Choose a reason for hiding this comment

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

high

사진 삭제 후 photos 쿼리만 무효화하고 있습니다. 이로 인해 다른 관련 데이터(예: 좋아요 목록, 앨범 정보 등)가 오래된 상태로 남아있을 수 있습니다. 사진 삭제 시 앨범과 관련된 모든 쿼리를 무효화하여 데이터 정합성을 보장하는 것이 좋습니다.

photoDetail 쿼리를 무효화하면 현재 상세 페이지가 자동으로 업데이트되어 '삭제된 사진'임을 사용자에게 알리는 로직을 구현할 수도 있습니다.

      queryClient.invalidateQueries({ queryKey: [EP.album.photos(albumId)] });
      queryClient.invalidateQueries({ queryKey: [EP.album.photoDetail(albumId, photoId)] });
      queryClient.invalidateQueries({ queryKey: [EP.album.likedPhotos(albumId)] });
      queryClient.invalidateQueries({ queryKey: [EP.album.albumBest-4cut(albumId)] });
      queryClient.invalidateQueries({ queryKey: [EP.album.availableCount(albumId)] });
      queryClient.invalidateQueries({ queryKey: [EP.album.albumInfo(albumId)] });

@dasosann dasosann merged commit 960c11e into main Jan 21, 2026
4 checks passed
@dasosann dasosann deleted the fix/betaQA2 branch January 21, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants