Skip to content
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

feat: 닉네임 변경 기능 구현 #569

Merged
merged 17 commits into from
Nov 16, 2023

Conversation

hyunji1203
Copy link
Collaborator

@hyunji1203 hyunji1203 commented Nov 16, 2023

📌 관련 이슈

🛠️ 작업 내용

  • 마이페이지 뷰 변경
  • 닉네임 뷰 생성
  • 마이페이지에서 보여주는 닉네임 profiles PATH에서 가져오도록 변경
  • 닉네임 변경 기능 구현
  • 닉네임 변경 후 바뀐 닉네임으로 마이페이지에 보여지도록 구현

🎯 리뷰 포인트

  • 디자인이 괜찮은지 궁금합니다.
  • 현재 확장성을 염두해두고 Profile 이라고 명칭을 하였습니다. (지금은 닉네임만 가능하지만 언젠가...훗날 가능할 프로필 사진 변경을 위해...)
  • 닉네임 변경 후 마이페이지로 돌아갔을 때 변경한 닉네임으로 화면에 보여지도록 activityResultLauncher를 사용했는데, 해당 로직을 잘 사용했는지 궁금합니다.
  • [확인 완료! 완벽 성공!] 서버에 프로필(닉네임)관련 기능이 붙어있지 않은 상황이라 실제로 잘 돌아가는지 아직 확인하지 못했습니다. 시간 절약을 위해 우선 PR 작성을 하였는데, 이따가 서버에 기능이 붙으면 잘 돌아가는지 확인해보도록 하겠습니다

⏳ 작업 시간

추정 시간: 4h
실제 시간: 8h
이유: 없는 뷰를 만들어야 해서 뷰 배치 및 디자인 고민이 은근히 시간이 걸렸습니다. 그리고 양방향 바인딩을 직접 처음 써보게 되었는데 이를 공부하면서 적용하느라 좀 시간이 걸렸습니다.

Copy link
Collaborator

@briandr97 briandr97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

@briandr97 briandr97 merged commit ce5d6a3 into dev_android Nov 16, 2023
3 checks passed
@briandr97 briandr97 deleted the feat/#568_닉네임_변경_기능_구현 branch November 16, 2023 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants