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

#217, #219 : 프로필 한줄소개, 이미지 업데이트 API integration 적용해요, Profile 엔티티 수정해요. #221

Merged
merged 18 commits into from
Jan 20, 2025

Conversation

jeongjaino
Copy link
Member

@jeongjaino jeongjaino commented Jan 14, 2025

1. 📄 관련된 이슈 및 소개

#217 : 프로필 한줄소개 / 이미지 업데이트 API integration 적용해요

2. 🔥 변경된 점

1. 프로필 한줄 소개와 이미지 업로드 두 API가 통합되었습니다.

  • 이전에 대면 회의에서 공유드렸다시피 둘 중 하나가 실패하면, 토스트가 중복으로 노출되는 이슈가 있었어요. 두 API 응답으로 모두 각각 처리했기 때문인데요,
  • 위 이슈위에도 하나의 API가 실패하면 둘 모두 실패하는 것으로 논의되어서, 서버에서 해당 API를 통합하고 저희도 하나만 호출하도록 변경했습니다.

2. NotificationSetting 패키지 변경

  • NotificationSettingState가 screen 패키지에 있어서, state로 옮겼습니다.

3. EntireEdit -> ProfileEdit 이름 변경 / ProfileEdit Arugment 삭제

  • 이전에는 EntireEditAction, State, SideEfffect가 3개 Screen을 담당했는데, 이제 하나로 바뀌면서 ProfileEdit으로 모두 바꿨습니다.
  • ProfileEdit Argument도 이제 사용을 안하게 되면서, 관련 코드 모두 삭제했습니다.

4. 투표 결과 패딩 수정

5. ProfileCharacter

  • 도메인 모델 내 주석추가했습니다잇 :찡긋

6. 승원아 loading 상태 변수 잘못쓰고 있어서, 지금까지 프로필 업데이트할때 로딩 로띠 안돌았다.

3. ✅ 필수 체크 사항

1. ProfileEditViewModel

한줄 소개 업데이트, 프로필 이미지 업데이트 로직 통합했는데, 해당 로직 확인 부탁해용 💯

ProfileEditViewModel handleProfileUpdate()

4. 📸 작업물 사진 공유(선택)

1. 프로필 업데이트

profile_edit1.webm

2. 프로필 업데이트 (사진 삭제)

profile_edit2.webm

5. 💡알게된 혹은 궁금한 사항

오랜만에 보니까 수정할만한 사항이 많네요 ㅎㅎ

@jeongjaino jeongjaino added 🍻제이콥🍻 24기 정진호 🔨픽스🔨 버그 및 잠재적 문제 해결 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요? labels Jan 14, 2025
@jeongjaino jeongjaino self-assigned this Jan 14, 2025
@jeongjaino jeongjaino changed the title #217 : 프로필 한줄소개 / 이미지 업데이트 API integration 적용해요 #217, #219 : 프로필 한줄소개, 이미지 업데이트 API integration 적용해요, Profile 엔티티 수정해요. Jan 15, 2025
@jeongjaino jeongjaino closed this Jan 15, 2025
@jeongjaino jeongjaino reopened this Jan 15, 2025
Copy link
Contributor

@flash159483 flash159483 left a comment

Choose a reason for hiding this comment

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

원래 맛있는건 천천히 먹어야지

@@ -64,6 +63,6 @@ class NotificationNavigatorImpl(private val navController: NavController): Notif
}

override fun navigateToProfileEditScreen() {
navController.navigate(ProfileEditScreenDestination(ProfileEditNavArgs(false)) within AppNavGraphs.entire)
navController.navigate(ProfileEditScreenDestination() within AppNavGraphs.entire)
Copy link
Contributor

Choose a reason for hiding this comment

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

Object면 가로 빼도됨

Copy link
Member Author

Choose a reason for hiding this comment

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

반영했습니다 👍

@jeongjaino jeongjaino merged commit 37d5a72 into develop Jan 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍻제이콥🍻 24기 정진호 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요? 🔨픽스🔨 버그 및 잠재적 문제 해결
Projects
None yet
2 participants