Skip to content
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

[FIX] 로그인, 동네인증 API response 수정 및 UX 로직 분기처리 (#27) #32

Merged
merged 19 commits into from
Feb 17, 2025

Conversation

yurim830
Copy link
Collaborator

@yurim830 yurim830 commented Feb 15, 2025

🐿️ Pull Requests

🪵 작업 브랜치

🥔 작업 내용

hasVerifiedArea(Bool)를 UserDefaults에서 관리합니다.

  • 저장: Login 성공 후 response를 저장
  • 사용:
    • 로그인 VC에서 - 다음 화면 결정
      • hasVerifiedArea ? SpotListVC : LocalVerificationVC
    • Splash에서 - (회원가입 시 동네 인증 안하고 강제종료했지만 자동로그인 된 경우 대응)
      • !hasVerifiedArea → LocalVerificationVC push

🚨 참고 사항

TODO: Splash에서 동네인증으로 보내는 경우 자동으로 맵뷰로 넘어가는 문제 있음.

📸 스크린샷

회원가입 후 동네인증 안 하고 강제종료했다가 다시 앱에 들어온 경우: SplashVC에서 동네인증으로 보냄

ScreenRecording_02-17-2025.17-34-14_1.MP4

동네인증 안 한 사람이 앱 삭제했다가 다시 로그인 한 경우: LoginVC에서 동네인증으로 보냄

ScreenRecording_02-17-2025.17-39-51_1.MP4

동네인증 한 사람이 로그아웃했다가 다시 로그인 한 경우: LoginVC에서 탭바로 보냄

ScreenRecording_02-17-2025.17-39-24_1.MP4

💥 To be sure

  • 모든 뷰가 잘 실행되는지 다시 한 번 체크해주세요 !

🌰 Resolve issue

@yurim830 yurim830 added ❤️‍🩹 bug 버그수정 🌀 feature 새로운 기능 개발 🥑 유림 유림 labels Feb 15, 2025
@yurim830 yurim830 added this to the Sprint-2 milestone Feb 15, 2025
@yurim830 yurim830 self-assigned this Feb 15, 2025
Copy link
Contributor

@cirtuare cirtuare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

조아용 ~~ 굿잡
서버 배포 확인하구 바로 머지해주세영

@yurim830 yurim830 changed the title [FIX] 로그인, 지역인증 API response 수정 및 UX 로직 분기처리 [FIX] 로그인, 지역인증 API response 수정 및 UX 로직 분기처리 (#27) Feb 16, 2025
Copy link
Collaborator

@Ohjackson Ohjackson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굳굳~~!

Copy link
Contributor

@cirtuare cirtuare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿

Comment on lines +103 to +111
case .profileEdit:
guard let vcStack = self?.navigationController?.viewControllers else { return }
self?.localArea = areaName
for vc in vcStack {
if let profileEditVC = vc as? ProfileEditViewController {
self?.navigationController?.popToViewController(profileEditVC.self, animated: true)
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

추후 삭제 ~~

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넹넹~~ 플로우 변경된 거 다른 브랜치에서 작업하면서 이거 삭제할게용

Merge branch 'develop' into fix/#27
@yurim830 yurim830 changed the title [FIX] 로그인, 지역인증 API response 수정 및 UX 로직 분기처리 (#27) [FIX] 로그인, 동네인증 API response 수정 및 UX 로직 분기처리 (#27) Feb 17, 2025
@yurim830 yurim830 merged commit e7c60c4 into develop Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❤️‍🩹 bug 버그수정 🌀 feature 새로운 기능 개발 🥑 유림 유림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 로그인, 동네인증 API response 수정 및 로직 분기처리
3 participants