-
Notifications
You must be signed in to change notification settings - Fork 1
fix : 시연 테스트중 발견한 이슈수정 #152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @caseBread, 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은 시연 테스트 중 발견된 문제를 해결하기 위해 앨범 목록 내에서 사진 개수와 좋아요 정보가 표시되는 방식을 개선합니다. 앨범 타입 인식을 도입하여 선택된 앨범 보기에 따라 UI가 콘텐츠를 정확하게 반영하도록 함으로써 데이터 일관성과 사용자 경험을 향상시킵니다. Highlights
Using Gemini Code AssistThe 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
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 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
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
시연 테스트 중 발견된 이슈를 수정하는 PR이네요. albumType 상태를 추가하여 사진 개수 표시와 '좋아요' 정보 표시 로직을 개선한 점이 좋습니다. 전반적으로 올바른 수정으로 보입니다. 다만, 사진 개수를 표시하는 부분에서 사용자 경험을 조금 더 개선할 수 있는 점이 있어 리뷰 코멘트를 남겼습니다. 확인 부탁드립니다.
| <div className='mb-3 flex justify-between'> | ||
| <span className='typo-body-lg-regular text-text-subtle'> | ||
| 총 {totalPhotoCount || 0}장 | ||
| 총 {(albumType === 'all' ? totalPhotoCount : photos.length) || 0}장 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
무한 스크롤이 적용된 리스트에서 '총'이라는 단어와 함께 현재 로드된 사진의 개수(photos.length)를 보여주는 것은 사용자에게 혼란을 줄 수 있습니다. 전체 사진 개수가 아닌 로드된 개수임을 명확히 하거나, '총'이라는 단어를 제거하는 것을 고려해보는 것이 좋겠습니다. 예를 들어, 'all' 타입일 때만 '총'을 붙여주고, 다른 경우에는 사진 개수만 표시할 수 있습니다.
| 총 {(albumType === 'all' ? totalPhotoCount : photos.length) || 0}장 | |
| {albumType === 'all' ? `총 ${totalPhotoCount || 0}장` : `${photos.length || 0}장`} |
요약
구현 사항
📸 스크린샷
Need Review
Reference
📜 리뷰 규칙
Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.