-
Notifications
You must be signed in to change notification settings - Fork 0
[TEST] 5차 스프린트 테스트 앱을 배포합니다. #313
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
Merged
Conversation
This file contains hidden or 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
- LanguageProviderImpl에서 locale 변수 가져오는 방식을 조금 더 가볍게 개선했습니다. 기존 방식은 앱 내 다국어 지원 기능이 있을때 효과적인 방법이라고 합니다.
- 기존 로직은 닉네임 설정 완료 후 다시 한번 카카오 sdk를 호출하여 토큰을 받아 회원가입 api에 사용했습니다. - 로그인 화면에서 카카오 sdk 호출 후 OAuthDataStore에 토큰을 저장하도록 수정하여 두 번 호출하는 로직을 수정하며, 구글 로그인과 로직을 비슷하게 수정하였습니다.
- 에러 메시지를 제공하는 ErrorMessageProvider 인터페이스와 구현체 ErrorMessageProviderImpl을 추가했습니다. - ErrorMessageModule을 추가하여 ErrorMessageProvider를 주입할 수 있도록 설정했습니다.
- airbridge 라이브러리 때문에 dataExtractionRules, fullBackupContent 부분에서 중복선언이 이루어져 replace를 확장했습니다.
- ViewModel에서 NetworkUtil 대신 NetworkConnectivityObserver 사용 - 에러 메시지를 ErrorMessages 객체 대신 ErrorMessageProvider를 통해 제공하도록 수정 - 각 ViewModel의 네트워크 요청 및 에러 처리 로직에 변경된 방식 적용
[Chore/#314] 5차 스프린트 QA 사항을 반영합니다.
[REFACTOR/#297] 에러 핸들링 중앙화 해버리기
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.
📌 ISSUE
closed #312
📄 Work Description
✨ PR Point
📸 ScreenShot/Video