#205 : 프로필 수정 푸시 클릭 동작 구현해요 #207
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jeongjaino
added
🌱기능🌱
2 tasks
flash159483
reviewed
Nov 25, 2024
| viewModelScope.launch { | ||
| autoLoginUseCase(versionCode).let { | ||
| autoLoginUseCase(state.versionName).let { |
There was a problem hiding this comment.
이걸 state에 저장해야될 이유가 있나요? uiState에 넣기에는 ui에는 아무런 영향을 주지 않다 보니깐 좀 애매한데요
There was a problem hiding this comment.
오 생각해보니까 그렇네요 !
UiState에서 사용되지 않는데, 굳이 담아둘 필요가 없는 것 같아요 !
다음에 앱 버전이랑 코드 같은 점들 캐싱해서 가져오면 좋을 것 같은데요, 일단 뷰모델 내 변수에 담아두는 방식으로 수정했습니다 !
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. 📄 관련된 이슈 및 소개
#205 : 프로필 수정 푸시 클릭 동작 구현해요
2. 🔥 변경된 점
다음 버전부터 서버에 앱 버전을 전송해요 !
로그인 / 회원가입시에 앱 버전을 전송해요,
푸시 알림 수신시, 프로필 설정화면으로 전환 구현했습니다 !
3. ✅ 필수 체크 사항
로그인 시 사용하는 모델 변경
기존 로그인시에 사용하는 도메인 모델이 fcmToken / fcmTokenDto였는데요 !
presentation 계층으로부터 받아온 앱 버전을 프로퍼티에 추가해야 해서, fcmToken 클래스에 들어가는건 부적절하다고 생각해서,
SignUp / SignUpDto 기반으로 새로 SignIn 모델을 새로 구성했어요 !
naivagetUp && 뒤로가기 동작 확인했습니다 !
4. 📸 작업물 사진 공유(선택)
회원가입 / 로그인시 적재되는 앱버전 확인했고, 서버에서도 정상적으로 적재되는 점 확인했습니다.
관련해서 로그 기록이랑 동영상 첨부드립니다 !
'
Screen_recording_20241124_035645.webm
5. 💡알게된 혹은 궁금한 사항