-
Notifications
You must be signed in to change notification settings - Fork 0
[FEAT] 친구 프로필 / 프로필 수정 개선 #45
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 @rhkrwngud445, 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은 친구 프로필 및 홈 화면의 사용자 경험을 개선하는 여러 변경 사항을 포함하고 있습니다. 전반적으로 코드 변경 사항들은 의도한 대로 잘 구현되었습니다. 친구 삭제 시 홈 화면에 즉시 반영되도록 하는 로직과 프로필 수정 화면에서 뒤로가기 동작을 통일한 점이 인상적입니다. 다만, 홈 화면의 에러 처리 방식과 네비게이션 그래프에 남아있는 불필요한 코드에 대해 몇 가지 개선점을 제안합니다.
| launch { | ||
| viewModel.errorEvent.collect { | ||
| onShowErrorSnackBar(it) | ||
| onShowErrorSnackBar(IllegalStateException("네트워크 에러가 발생했습니다.")) |
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.
기존에 viewModel.errorEvent에서 전달되던 Throwable 객체를 사용하지 않고, 새로운 IllegalStateException을 생성하여 하드코딩된 메시지를 전달하고 있습니다. 이 방식은 원래 발생했던 에러의 상세 정보를 잃게 만들어 디버깅을 어렵게 할 수 있습니다. 기존 구현처럼 onShowErrorSnackBar(it)을 그대로 호출하여 원래의 Throwable을 전달하는 것이 더 바람직해 보입니다. onShowErrorSnackBar의 구현을 보면 null이거나 message가 없는 경우를 이미 잘 처리하고 있습니다.
| onShowErrorSnackBar(IllegalStateException("네트워크 에러가 발생했습니다.")) | |
| onShowErrorSnackBar(it) |
| friendId?.let { | ||
| viewModel.deleteFriend(it) | ||
| } | ||
| friendId |
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.
|
고생하셨습니다! |
…ile' into feat/#43-improvement_friend_profile
작업 내용
친구 프로필 상세
친구 프로필 수정
홈 화면
확인 방법
홈 / 친구 프로필 / 친구 프로필 수정
참고 사항
N/A
관련 이슈