-
Notifications
You must be signed in to change notification settings - Fork 0
[#51] 코디 꾸미기 수정 및 api 1차 연결 #58
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
Summary of ChangesHello @Funital, 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은 앱의 핵심 기능인 홈, 룩북, 알림, 검색 모듈에 대한 API 연동을 광범위하게 구현하고, 데이터 모델을 표준화하며, 의존성 주입 구조를 개선하는 데 중점을 둡니다. 이를 통해 실제 백엔드 서비스와의 통합을 강화하고, 사용자 경험을 향상시키기 위한 기반을 마련합니다. 또한, 코드의 유지보수성과 확장성을 높이는 리팩토링 작업도 포함되어 있습니다. 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. Changelog
Activity
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. API 연결, 새 길을 열어, 데이터 흐름, 앱에 생명 불어넣네. 코드의 춤, 기능의 노래. 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은 목(Mock) 데이터를 실제 API로 전환하는 중요한 작업을 포함하고 있네요. DI 컨테이너 리팩토링, ID 타입 표준화, 새로운 DTO 추가 등 코드 베이스를 크게 개선하는 많은 노력이 엿보입니다. 전반적으로 훌륭한 작업이지만, 몇 가지 잠재적인 문제점과 개선 사항을 발견하여 리뷰 코멘트를 남겼습니다. 특히, 일부 기능이 주석 처리되어 동작하지 않는 부분과 하드코딩된 값, Swift 네이밍 컨벤션 위반 사항들을 확인해주시면 좋겠습니다.
🔗 연결된 이슈
Resolved #51
✨ 주요 작업사항
실제 API 서비스를 데이터 소스에 통합하고, 의존성 주입(DI) 컨테이너를 리팩토링하며, Home 관련 API 요청·응답을 위한 새로운 DTO를 추가함으로써 Home 및 LookBook 모듈에 중요한 개선 사항을 도입합니다. 또한 LookBook DI 컨테이너 전반에서 ID 타입을 Int64로 표준화하고, 새로운 문자열 리터럴을 추가해 로컬라이제이션을 강화했으며, 몇 가지 소소한 개선과 디버그 유틸리티도 포함되어 있습니다.
가장 중요한 변경 사항은 다음과 같습니다:
1. Home 모듈 API 연동 및 리팩토링
이를 통해 기존의 목(Mock) 데이터 로직을 대체했습니다.
2. LookBook 모듈 리팩토링 및 타입 표준화
3. 모듈 전반의 의존성 주입 개선
4. 로컬라이제이션 및 사용자 피드백 개선
5. 기타 개선 사항 및 디버그 유틸리티
📸 스크린샷 / 동영상
🔍 리뷰어에게 (선택)
우선 사용해야 하는 모든 api 에 대해 sevice, datasource, repository, repositoryImpl, usecase까지는 모두 작성 완료했고 view와 viewModel에 적용만 남은 상태입니다.