[Feat] #225 - 강제업데이트 추가 및 로그인 오류 개선 #228
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
jeongdung-eo
approved these changes
Jan 28, 2024
Comment on lines
+40
to
+55
| Task { | ||
| do { | ||
| let updateType = try await checkUpdateType() | ||
| switch updateType { | ||
| case .force(let newVersion): | ||
| self.showForceUpdateAlert(newVersion: newVersion) | ||
| case .optional: | ||
| self.showUpdateAlert() | ||
| case .none: | ||
| self.changeMainViewController() | ||
| } | ||
| } catch { | ||
| self.changeMainViewController() | ||
| } | ||
| } | ||
| } |
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.
🫧 작업한 내용
진입 로직은
앱 실행 -> CheckUpdate뷰컨(업데이트 체크 및 alert띄우기) -> 토큰정보 있으면 홈으로, 없으면 온보딩으로
입니다!!!!
업데이트 URL이 자꾸 유효하지 않은 URL로 떠서 원래는 사파리 실행 URL이었지만 앱스토어 앱이 바로 열리도록 스킴(itms-어쩌구로)을 바꿨습니다! 근데 이게 시뮬에는 앱스토어 앱이 없어서 테스트가 안돼서.. 윤서가 여행에서 돌아오면 인증서 파일을 받아서 테스트해볼게여. . 저번에 프로비저닝만 받고.. 인증서를 안받음 ㅎ
테스트하다보니까 로그인 안되는 부분을 찾아서 추가했습니다~ 저 알림이 첨에는 잘 뜨는데 그 이상은 break로 빠져버려서 홈으로 진입하는 루트가 없어서 그랬던 거였어요~ 추가해놧슴당
컨벤션은 다른 파일들보고 최대한 맞게 한다고 해놓긴 했는데 완벽숙지가 된 상태는 아니라.. 혹시 틀린부분ㄴ 있으면 알려주세열..ㅎ.ㅎ
🔫 PR Point
📸 스크린샷
근데 갠적으로 문구가 뭔가 강제로 하는 느낌이 아니라서 "더이상 지원되지 않는 버전입니다" 이런게 들어가야 할것 같긴 하네여.. 담에 기획에게 물어보겟슴다..
📮 관련 이슈