-
Notifications
You must be signed in to change notification settings - Fork 0
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] 온보딩 / 가이드, 스토리, 닉네임 기능 구현 #114
Conversation
…o feature/ui-onboarding-guide
…o feature/ui-onboarding-guide � Conflicts: � app/src/main/AndroidManifest.xml
…b.com/team-winey/Winey-AOS into feature/ui-onboarding-guide � Conflicts: � app/src/main/AndroidManifest.xml
…b.com/team-winey/Winey-AOS into feature/ui-onboarding-guide
…com/team-winey/Winey-AOS into feature/ui-onboarding-guide � Conflicts: � app/src/main/java/com/android/go/sopt/winey/presentation/splash/SplashActivity.kt
…o feature/ui-onboarding-guide # Conflicts: # app/src/main/res/layout/fragment_level_guide.xml
…-guide # Conflicts: # app/src/main/java/com/android/go/sopt/winey/presentation/splash/SplashActivity.kt # app/src/main/res/layout/activity_guide.xml
- popBackStack 명시적으로 호출하기
…o feature/ui-onboarding-guide � Conflicts: � app/src/main/java/com/android/go/sopt/winey/data/repository/AuthRepositoryImpl.kt � app/src/main/java/com/android/go/sopt/winey/data/service/AuthService.kt � app/src/main/java/com/android/go/sopt/winey/data/source/AuthDataSource.kt � app/src/main/java/com/android/go/sopt/winey/domain/repository/AuthRepository.kt � app/src/main/java/com/android/go/sopt/winey/presentation/main/feed/upload/amount/AmountViewModel.kt
…o feature/ui-onboarding-guide � Conflicts: � app/src/main/java/com/android/go/sopt/winey/presentation/splash/SplashActivity.kt
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.
뷰가 많아서 힘들었을텐데 고생하셨습니다 !! 어푸 찍어놓을게용 !!👏👏👏👍👍👍
} | ||
} | ||
} | ||
onBackPressedDispatcher.addCallback(this, callback) |
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.
오 이런식으로 뒤로가기 이벤트 설정이 가능하군요 👏👏
@@ -53,7 +53,7 @@ class NicknameActivity : BindingActivity<ActivityNicknameBinding>(R.layout.activ | |||
|
|||
private fun initCompleteButtonClickListener() { | |||
binding.btnNicknameComplete.setOnClickListener { | |||
// todo: 서버에 닉네임 PATCH | |||
viewModel.patchNickname() | |||
navigateTo<MainActivity>() |
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.
patchNickname()이 onSuccess되었을때만 MainActivity로 넘어가게하는건 어떤가요? 중복확인후 닉네임변경 버튼을 누르는 사이에 다른 유저가 해당 닉네임으로 변경했을경우 409 에러(닉네임 중복) 가 뜰 수도 있을것 같습니다 !
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.
뷰모델에서 서버통신 결과를 가져와서 그에 따라 UI를 갱신하는 코드가 빠져있네요! 좋은 지적 감사합니다 :)
…-winey/Winey-AOS into feature/ui-onboarding-guide � Conflicts: � app/src/main/res/layout/fragment_first_story.xml � app/src/main/res/layout/fragment_second_story.xml � app/src/main/res/layout/fragment_third_story.xml
…o feature/ui-onboarding-guide # Conflicts: # app/src/main/java/com/android/go/sopt/winey/presentation/splash/SplashActivity.kt
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.
👍👍👍👍👍👍👍👍👍👍
📝 Work Description
📣 To Reviewers