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

[Fix] 최종 QA 반영 #395

Merged
merged 2 commits into from
Sep 23, 2024
Merged

[Fix] 최종 QA 반영 #395

merged 2 commits into from
Sep 23, 2024

Conversation

hooni0918
Copy link
Member

🔗 연결된 이슈

📄 작업 내용

  • 어쩌구저쩌구
구현 내용 IPhone 15 pro
GIF

💻 주요 코드 설명

설명할 코드 주제 또는 기능

MyPageEditViewController:

기존 publisher로 전달주던 코드에서 selectedImage 변수를 추가하여 선택된 이미지를 임시로 저장하게 수정했습니다
handleConfirmButtonTap 메소드를 추가하여 '수정하기' 버튼을 눌렀을 때만 이미지를 업데이트 하게 수정했습니다
그리고 이미지 선택 시 rootView.profileImageView.image를 업데이트합니다
'수정하기' 버튼을 활성화하는 로직을 추가했습니다.

MyPageEditViewModel:

newProfileImage 옵저버블 대신 updateProfileImage 메소드를 직접 호출하도록 변경했습니다.
setDefaultProfileImage 메소드를 추가하여 기본 프로필 이미지 설정을 처리합니다.

해당 코드가 있는 파일명

@hooni0918 hooni0918 added ✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면 labels Sep 22, 2024
@hooni0918 hooni0918 self-assigned this Sep 22, 2024
Copy link
Member

@youz2me youz2me left a comment

Choose a reason for hiding this comment

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

고생하셨습니닷 ㅎㅎ

궁금한게 있는데 저 프로필 기프티콘 쓰신건가요? 아직 안쓰셨으면 제가 쓰겠습니다.

Copy link
Member

@mmaybei mmaybei left a comment

Choose a reason for hiding this comment

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

굿! 수고하셨습니다

Copy link
Contributor

@JinUng41 JinUng41 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

@hooni0918 hooni0918 merged commit 33adacc into suyeon Sep 23, 2024
@hooni0918 hooni0918 deleted the fix/#394-qaSolveJihoon branch September 23, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants