-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 로그인 v2 api 작업 #437
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?
[Feat] 로그인 v2 api 작업 #437
Conversation
Summary of ChangesHello @HI-JIN2, 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은 로그인 기능을 v2 API로 업데이트하고, DeviceType 정보를 추가하여 로그인 요청을 처리하도록 변경합니다. 또한, 기존 로그인 API 관련 코드에서 data, domain, presentation 레이어 간의 의존성 문제를 해결하기 위해 DTO 클래스가 domain 및 presentation 레이어에서 직접 사용되지 않도록 리팩토링하여 아키텍처의 견고성을 높였습니다. 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은 로그인 API를 v2로 업데이트하고 DeviceType을 추가하는 작업을 포함하고 있습니다. 동시에 데이터, 도메인, 프리젠테이션 레이어 간의 의존성 문제를 해결하기 위해 DTO가 도메인 및 프리젠테이션 레이어에서 사용되는 부분을 리팩토링한 점이 인상적입니다. 전반적으로 클린 아키텍처 원칙을 잘 따르려는 좋은 시도이며, 코드 변경 사항은 명확하고 목적에 부합합니다. 몇 가지 사소한 스타일 관련 제안 사항을 리뷰 코멘트로 남겼습니다.
app/src/main/java/com/eatssu/android/data/remote/dto/request/LoginWithKakaoRequest.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/eatssu/android/domain/usecase/auth/LoginUseCase.kt
Outdated
Show resolved
Hide resolved
core/common/src/main/java/com/eatssu/common/enums/DeviceType.kt
Outdated
Show resolved
Hide resolved
…oginWithKakaoRequest.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…UseCase.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
PeraSite
left a comment
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.
LoginWithKakaoRequest처럼 dto 클래스를 도메인 내부에서 생성해서 넘겨주는 흐름이 이제 보니 되게 많네요!
코드베이스의 다른 부분에서도 점진적으로 개선해야겠네용!
Summary
로그인 v2 api 작업
Describe your changes
구체적으로는 dto class가 domain과 presentation에서 쓰이는 것을 제거했습니다.
(타 api 관련 리팩토링은 별도의 pr로 분리하겠습니다)
Issue
To Reviewer
이 부분에 대해 코멘트 해주시면 감사하겠습니다!