[TNT-221] 전체 화면 흐름 관련 보완 로직 작성 #69
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
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.
📌 What is the PR?
🪄 Changes
🔥 PR Point
NetworkService의 요청/응답/에러 발생 시의 Notification 처리를 보완했습니다. 물론 리팩토링해야할 사안이기는 하지만, 현재 네트워크 로직 상에서 에러 발생시 progress hide 처리가 되지 않고 있고, 세션 만료인 401 발생 시 라우팅하는 Notification도 없었기 때문에, 해당 내용을 작성했습니다.
AppLink와 AppStorage를 추가했습니다. 각각 앱에서 외부로 연결되는 링크, 앱내에서 저장/관리되는 값을 의미합니다. 해당 부분은 나중에 같이 얘기해보면 좋을 것 같아요.
보완된 NetworkService 로직에 더불어 AppFlowCoordinator에 초기 스플래시와 세션 유효확인, 세션 만료시 로그인 화면 라우팅 처리를 작성했습니다. 이제 앱 진입 시 스플래시가 표시되며 - 동시에 세션 유효성을 확인합니다. 앱 어느 화면에서든 API 호출 후 401이 발생하면 팝업과 함께 SNS로그인 화면으로 라우팅됩니다.
앱 아이콘을 추가했습니다. 이뻐짐!
📸 Screenshot
🙆🏻 To Reviewers
💭 Related Issues