[Fix] 4.1.0 출시 검증에서 나온 즉시 수정 4건 - #282
Merged
Merged
Conversation
…이 실제로 계정을 지우고 소셜 연동 해제도 실제로 도는데, 카카오 unlink 만 예외를 감싸지 않아 실패하면 서버 탈퇴까지 멈췄다, 마이페이지의 두 콜백은 async 를 VoidCallback 자리에 넘겨서 예외를 아무도 받지 못해 화면은 그대로고 Sentry 에만 fatal 로 쌓였다, 구글 애플처럼 카카오도 실패를 삼켜 탈퇴를 이어가고 화면은 실패를 알린 뒤 넘어가지 않는다
… 방이 남는데 번호 확인이 로딩 중에만 걸려 있었다, 제목과 멤버는 앞 방인데 나가기와 삭제는 이 화면의 방 번호로 나가서 보고 있지도 않은 방에서 나가게 된다, 번호가 다르면 없는 것으로 보고 불러오지 못했다고 알린다
…t 사이에 화면이 살아 있는지 보지 않았다, 세트를 받는 동안 나가서 다른 세트를 열면 늦게 끝난 쪽이 공용 상태를 덮어 화면에 떠 있는 세트에 다른 세트의 문제가 들어가고 복습 횟수도 엉뚱한 세트에 올라간다, 화면이 사라졌으면 이어가지 않는다
…그 전에 들어온 두 번째 탭은 그대로 콜백을 탔다, 복습 횟수와 경험치가 두 번 오르는 자리라 눌린 순간에도 확인한다
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
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.
✔️ 연관 이슈
📝 작업 내용
4.1.0 출시 전 검증(정적 분석, 테스트, 성능·로직·부작용·권한 검토)에서 나온 즉시 수정 항목 3건과, 확인 요청받은 복습 완료 중복 저장 1건을 고쳤습니다. 검토 범위는 4.0.0 출시 감사 이후(
c84d1b3..HEAD) 83커밋입니다.1. 탈퇴·로그아웃 실패 경로 (
KakaoAuthService,MyPageScreen)loginMethod대소문자를 맞추면서, 그동안 한 번도 안 걸리던 게스트 분기와 소셜 연동 해제가 이제 실행됩니다. 게스트 로그아웃이 진짜로 서버 계정을 지웁니다.deleteAccount가 서버 삭제에 닿지 못하고 던졌습니다. 구글·애플처럼 실패를 삼키고 탈퇴를 이어갑니다.VoidCallback자리에async클로저를 넘겨서 반환 Future 를 버리고 있었습니다. 실패를 받아 안내하고 화면을 넘기지 않습니다.2. 스터디룸 상세의 방 번호 확인 (
StudyRoomDetailScreen)leaveRoom)와 삭제는 이 화면의 방 번호로 나갔습니다. 방장 메뉴도 앞 방 기준으로 열렸습니다.3. 복습 세트 로더의 생존 확인 (
PracticeDetailLoader)mounted확인이 없었습니다: 세트를 받는 동안 나가서 다른 세트를 열면, 늦게 끝난 쪽이moveToPractice로 공용 상태를 덮었습니다.4. 복습 완료 중복 저장 (
PracticeCompletionScreen)_submitting으로 버튼을 잠그고 있었지만 판정이 빌드 시점이라, 다시 그리기 전에 들어온 두 번째 탭은 그대로 콜백을 탔습니다. 눌린 순간에도 확인합니다.✅ 검증
flutter analyze: error 0, warning 0 (develop 대비 새로 늘어난 info 없음)flutter test: 2164개 통과, 57개 skip, 실패 0 (새 테스트 5개 포함)🙏 전달사항
검증에서 나온 🟠 항목은 이번에 넣지 않았습니다. 따로 이슈로 뺄지 정하면 좋겠습니다. 큰 것만 적으면,