[REFACTOR] 저장 확인 다이얼로그 제거

[stopstone]

작업 내용

• 친구 프로필 편집 화면에서 저장 버튼 클릭 시 나타나던 확인 다이얼로그를 제거하고, 즉시 저장되도록 수정했습니다.

확인 방법

-feature/friendprofileedittor에서 확인하실 수 있습니다.

참고 사항

관련 이슈

[gemini-code-assist]

Summary of Changes

Hello @stopstone, I'm Gemini Code Assist¹! 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!

이 Pull Request는 친구 프로필 편집 화면의 사용자 경험을 개선하기 위해 저장 확인 다이얼로그를 제거하는 리팩토링을 수행합니다. 기존에는 프로필 편집 후 저장 버튼을 누르면 한 번 더 확인하는 다이얼로그가 나타났지만, 이제는 이 단계를 생략하고 즉시 저장되도록 변경하여 사용자가 더 빠르고 직관적으로 작업을 완료할 수 있도록 합니다. 이는 불필요한 인터랙션을 줄여 전반적인 앱 사용 흐름을 간소화하는 데 기여합니다.

Highlights

• 저장 확인 다이얼로그 제거: 친구 프로필 편집 화면에서 저장 버튼 클릭 시 나타나던 SaveConfirmDialog가 완전히 제거되었습니다.
• 즉시 저장 기능 구현: 저장 버튼 클릭 시 더 이상 확인 다이얼로그를 거치지 않고, 즉시 onSubmit() 함수를 호출하여 변경 사항이 저장되도록 수정되었습니다.
• 관련 코드 정리: FriendProfileEditorScreen.kt 파일에서 SaveConfirmDialog 관련 import, 상태 변수 (saveConfirmDialogState), 그리고 다이얼로그를 표시하던 로직이 모두 삭제되었습니다.
• 다이얼로그 파일 삭제: SaveConfirmDialog.kt 파일 자체가 프로젝트에서 제거되었습니다.

Using Gemini Code Assist

The 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 /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.

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

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

이 PR은 친구 프로필 편집 화면에서 저장 버튼을 눌렀을 때 나타나는 확인 다이얼로그를 제거하는 리팩토링을 수행합니다. 이로 인해 사용자는 추가적인 확인 단계 없이 즉시 변경 사항을 저장할 수 있게 되어 사용자 경험이 개선됩니다. 전체적인 변경 사항은 명확하며, 의도한 대로 잘 구현되었습니다. 다만, 이번 변경과 관련하여 유효성 검사 로직에서 발견된 잠재적인 문제에 대한 피드백을 하나 남겼습니다. 확인 부탁드립니다.

[gemini-code-assist]

onSubmit을 직접 호출하도록 변경한 것은 좋습니다. 하지만 이로 인해 FriendProfileEditorViewModel의 유효성 검사 로직에 있는 잠재적인 문제가 드러날 수 있습니다.

onSubmit 함수는 기념일 제목에 대해 isBlank()를 사용하여 유효성을 검사하지만, UI에 오류를 표시하는 error 상태는 onAnniversaryTitleChanged 함수에서 isEmpty()를 사용하여 설정됩니다. 이 불일치로 인해 사용자가 공백만 입력할 경우 UI에는 오류가 표시되지 않지만 저장은 조용히 실패하게 됩니다.

일관된 동작을 보장하고 사용자에게 명확한 피드백을 제공하기 위해, FriendProfileEditorViewModel의 onNameChanged 및 onAnniversaryTitleChanged에서도 isBlank()를 사용하여 유효성을 검사하고 error 상태를 설정하는 것을 강력히 권장합니다. 이렇게 하면 사용자가 저장 버튼을 눌렀을 때 왜 저장이 안 되는지 명확히 알 수 있습니다.