-
Notifications
You must be signed in to change notification settings - Fork 0
내부 테스트 피드백 수정 #278
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
내부 테스트 피드백 수정 #278
Conversation
Summary of ChangesHello @pinocchio22, 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/UX 측면에서 일부 시각적 요소와 텍스트 표시 방식이 개선되었습니다. 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
내부 테스트 피드백을 반영한 수정사항 잘 보았습니다. 전반적으로 API 변경사항(날짜 및 커서 타입 변경)이 여러 파일에 걸쳐 일관성 있게 적용되었고, 로그인/회원가입 플로우 개선, 검색 기능 강화 등 다양한 기능 개선 및 버그 수정이 이루어졌네요.
몇 가지 추가 개선 사항을 제안합니다.
- FCM 토큰 업데이트 로직에서 발생할 수 있는 버그 수정
- 회원 탈퇴 시 FCM 토큰 삭제 로직 재검토
- 일부 함수의 네이밍 개선 및 오타 수정
- Reactor 패턴에서 상태 업데이트가 누락될 수 있는 부분 수정
자세한 내용은 각 파일에 남긴 리뷰 코멘트를 참고해주세요.
MLS/Domain/Domain/UseCaseImpl/AuthAPI/LoginWithKakaoUseCaseImpl.swift
Outdated
Show resolved
Hide resolved
MLS/Presentation/DictionaryFeature/DictionaryFeature/DictionaryList/DictionaryListReactor.swift
Show resolved
Hide resolved
MLS/Presentation/BookmarkFeature/BookmarkFeature/BookmarkList/BookmarkListReactor.swift
Outdated
Show resolved
Hide resolved
…LS-iOS into refactor/#273-Test-Feedback # Conflicts: # MLS/Presentation/BookmarkFeature/BookmarkFeature/BookmarkList/BookmarkListViewController.swift
📌 이슈
✅ 작업 사항
@pinocchio22