-
Notifications
You must be signed in to change notification settings - Fork 0
[#29] Data(리포트) View, Viewmodel 구현 #40
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
base: develop
Are you sure you want to change the base?
Conversation
Summary of ChangesHello @taebin2, 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은 사용자의 옷장 데이터를 시각적으로 분석하고 보고하는 기능을 구현합니다. 월별 리포트 뷰를 중심으로, 카테고리별 선호 아이템, 아이템별 사용 수량, 그리고 전체 옷장 활용도를 도넛 및 막대 차트를 통해 직관적으로 제공하여 사용자가 자신의 의류 소비 패턴을 쉽게 파악할 수 있도록 돕습니다. Highlights
🧠 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 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은 월간 리포트 관련 뷰와 뷰모델을 구현하는 내용이네요. 재사용 가능한 DonutChartView 컴포넌트를 만들어 다양한 통계 뷰에 적용한 점이 인상적입니다. 전반적으로 코드 구조가 좋지만, 몇 가지 개선점을 제안합니다. 주요 내용은 매직 넘버 및 색상 값 처리, 코드 중복 제거, 그리고 아키텍처 개선에 대한 것입니다. 자세한 내용은 각 파일에 남긴 코멘트를 확인해주세요.
| case wardrobeFavorite(items: [CategoryFavoriteItem]) | ||
| case wardrobeItemStats(stats: [ItemUsageStat]) | ||
| case wardrobeUsage(stats: WardrobeUsageStat) |
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.
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.
해결을 하긴 했는데 원래 View랑 Viewmodel만 하려고 햇었는데 이것떄문에 애매해져서 흠 나중에 어짜피 다 수정했어햐하니까 된건지 잘 모르겠네요
🔗 연결된 이슈
Resolved #이슈번호
✨ 주요 작업사항
📸 스크린샷 / 동영상
🔍 리뷰어에게 (선택)