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

로그인 뷰 작은 화면 대응, 강제 업데이트 추가 및 로그인 관련 디버깅 #174

Merged
merged 18 commits into from
Feb 22, 2024

Conversation

Eunice0927
Copy link
Collaborator

개요

변경 사항

  • ContentView에서 SettingView가 탭에 더이상 노출되지 않도록 주석처리
  • 로그인 뷰 UI SE, Mini, Pro Max 대응
  • SettingView와 SettingRowView이름 변경(FirebaseTestView)
  • 앱 강제 업데이트 기능 추가
    • 앱스토어를 bundleID로 찾아 버전을 비교한다.
    • ContentView가 뜨면 버전을 체크 해 필요시 알림을 띄운다.
  • 로그인 뷰에서 오류 발생시 알림 처리

공유사항(배운 것, 참고하면 좋을 것)

  • UIApplication.applicationIconBadgeNumber is deprecated
    • 앱 알림 뱃지 deprecated 관련
  • Apple store bundle id 찾기
  • [iOS] Swift 강제 업데이트 Alert 설정
    • 버전 숫자 세 자리 중 세 번째의 사소한 패치때는 업데이트 하지 않을 수 있도록 하는 방법
  • App Store 업데이트 버전 체크
    • SwiftUI에서 앱 버전 확인해 강제 업데이트
  • Apple store bundle id 찾기
    • 앱스토어 업데이트 팝업 테스트 해 보고 싶다면 이 블로그의 내용을 참고해 번들 id와 앱 id를 업데이트 하세요.
  • 앱 버전(version) 관리 규칙 공부하기!~ 4.2.1 은 어떤 방식으로?
    먼저 가장 먼저 나오는 숫자는 주 Major 역할을 합니다
    기존 버전과 호환 되지 않으면서 대규모 업데이트를 의미한다고 합니다! 
    4.2.1 의 버전은 그동안 4번의 대규모 업데이트를 진행했다는 것을 알 수 있습니다! 
    그리고 버전 up 을 하는 경우 뒤 숫자들은 0으로 초기화 합니다!
    
    그 다음 두번째 나오는 숫자는 부 Minor 입니다
    기존 버전과 호환은 되지만 새로운 기능이 추가되는 경우 해 당 숫자가 증가됩니다! 
    4.2.1 의 버전에서 는 총 두번에 새로운 기능이 추가 되었다는 것을 알 수 있습니다! 
    
    마지막으로 나오는 숫자는 수 Patch 입니다.
    보통 버그를 수정하거나 여러 오류들을 수정하고 배포할 때 해당 숫자가 증가합니다!
    

스크린샷

제목 작업 전 작업 후
업데이트 팝업
로그인뷰 스크린샷 2024-02-22 오후 8 06 48

전달사항

@Eunice0927 Eunice0927 added 🎨 Design 뷰 디자인 작업을 했어요 ✨ Feat 새로운 기능을 구현했어요 labels Feb 22, 2024
@trafico-bot trafico-bot bot added 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 🎄 현진 Pull Request Reviews assigned to GitHub User: hjsupernova 🍖 주원 Pull Request Reviews assigned to GitHub User: lm-loki labels Feb 22, 2024
Copy link
Collaborator

@qlrmr111 qlrmr111 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
전 갠적으로 업뎃으로 앱 진입 막아버리는거 매우 불쾌한 경험이었는데 그전 버전 이용자 대응 안할려면 어쩔수없나 보네요 ㅠㅠ

@trafico-bot trafico-bot bot added 👍 Merge 진행 PR 승인이 완료되어 merge할 수 있어요 and removed 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 labels Feb 22, 2024
@trafico-bot trafico-bot bot added 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 and removed 👍 Merge 진행 PR 승인이 완료되어 merge할 수 있어요 labels Feb 22, 2024
@Eunice0927
Copy link
Collaborator Author

고생하셨습니다! 전 갠적으로 업뎃으로 앱 진입 막아버리는거 매우 불쾌한 경험이었는데 그전 버전 이용자 대응 안할려면 어쩔수없나 보네요 ㅠㅠ

저도 좀 안좋아하긴 해요...
일단 강제업데이트를 1차에 심어놔야 나중에 이 기능을 끌 수도 있는거니까 우선 심어보려구요!

@Eunice0927 Eunice0927 merged commit 4b967fd into dev Feb 22, 2024
@trafico-bot trafico-bot bot added the 👏 Merge 완료 PR이 성공적으로 merge되었어요 label Feb 22, 2024
@Eunice0927 Eunice0927 deleted the Login/FixUI branch February 22, 2024 16:51
@trafico-bot trafico-bot bot removed the 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 label Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍖 주원 Pull Request Reviews assigned to GitHub User: lm-loki 🎄 현진 Pull Request Reviews assigned to GitHub User: hjsupernova 🎨 Design 뷰 디자인 작업을 했어요 ✨ Feat 새로운 기능을 구현했어요 👏 Merge 완료 PR이 성공적으로 merge되었어요
Projects
None yet
2 participants