[FIX] 2차 QA 대응 및 Amplitude 설정 (#210) #211
Merged
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.
🐿️ Pull Requests
🪵 작업 브랜치
🥔 작업 내용
악 앰플 안 붙인 게 생각나서 마저 붙이고 다시 push할게요!!
그런데 해당 로직이
scrollViewWillEndDragging에 설정되어 있어, 손가락을 떼어야만 UI가 업데이트되었습니다.스크롤 시, 연속적으로 판단하고 UI 업데이트가 되도록
scrollViewDidScroll로 옮겼습니다.서버 명세서가 잘못 작성되었는데, 제가 그걸 기준으로 판단했어서 Header 높이가 잘못 적용되었습니다.
해당 부분 수정하여 이제 WebView가 잘 present 됩니다! (+ 폰트 설정이 잘못되어있어 수정했습니다)
🚨 참고 사항
장소 상세 모달 뒤 dim 진하게, 뒤에 뷰 채도 빠지는 문제 해결 X
BlurEffect가 적용된
UIVisualEffectView가 present 될 경우, 포커싱을 위해 iOS system 자체에서 뒷 배경의 채도가 빠진다고 합니다. 그리고 dim 또한 지금 시스템에서 자동으로 넣어지고 있긴 합니다.dim 진하기와 채도 빠지는 문제 해결을 위해서는 커스텀을 해야하는데 리소스 부족 + 애니메이션 부자연스러운 문제 때문에 수정 진행하지 않았습니다.
로그인 2번 시도해야 앱 진입되는 문제
지금은 재현이 안 되네요.. 어젯밤에 토큰 유효기간 주기가 짧았을 때 재현이 가능했던 것 보면, 토큰 문제일 가능성이 높아보여요.
로그상으로는 첫 번째 시도에서 postLogin API 통신이 아예 안 된 걸 보면, 여기 guard문 어디에선가 막혔던 게 아닐까 싶습니다.
refreshTokenIfNeeded 부분에서 막혔던 것 같습니다
💥 To be sure
🌰 Resolve issue