Skip to content

Conversation

@thingineeer
Copy link
Collaborator

@thingineeer thingineeer commented Feb 7, 2024

🌱 작업한 내용

  • firebase Remote Config 호출 시점 Combine으로 처리
  • 스플래쉬에서 인증하기전, Remote Config 조건에 따라 알림을 띄울건지, 인증 절차로 넘어갈건지 판단

🌱 PR Point

  • 마이너 버전 바뀔 때마다 호출을 하게 되면 불필요한 이벤트가 발생하기 때문에 메이저 버전에 한정 지어서 설정하였습니다.
    • "강제" 업데이트로 변경
  • 컴바인 으로 willEnterForegroundNotification 부분 처리를 해주었습니다.

📸 스크린샷

구현 내용 스크린샷
ex. 로그인 화면

📮 관련 이슈

  • Resolved: #이슈번호

@thingineeer thingineeer added Setting 프로젝트 세팅, 라이브러리 설치 명진😼 labels Feb 7, 2024
@thingineeer thingineeer requested a review from 513sojin February 7, 2024 14:17
@thingineeer thingineeer self-assigned this Feb 7, 2024
@thingineeer thingineeer changed the title #253 업데이트 전 확인 브랜치 [Setting] #253 - Remote Config 세팅 작업을 완료 하였습니다. Feb 7, 2024
Comment on lines +171 to +177
private func setObserver() {
NotificationCenter.default.publisher(for: UIApplication.willEnterForegroundNotification)
.sink { [weak self] _ in
self?.setRemoteConfig()
}
.store(in: cancelBag)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

앗..!!!!!!!!!!!!! 이걸 못떠올려서 splash에서 처리가 힘들었었는데 .. 천재만재 만만재 이명진 ㅠㅠ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

구웃

Copy link
Collaborator

@513sojin 513sojin left a comment

Choose a reason for hiding this comment

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

willEnterForegroundNotification 배워갑니다 ,, 고생하셨어요 !!

Comment on lines +124 to +127
let splitCurrentVersion = currentVersion.split(separator: ".").map { $0 }
let splitStoreVersion = storeVersion.split(separator: ".").map { $0 }

if splitCurrentVersion[0] < splitStoreVersion[0] {
Copy link
Collaborator

Choose a reason for hiding this comment

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

버전 앞 자리만 비교해서 업데이트 팝업 띄우는 부분 확인했습니다 ~

@thingineeer thingineeer merged commit 7f90674 into Runnect:develop Feb 7, 2024
@thingineeer thingineeer deleted the #253---업데이트-전-확인-브랜치 branch February 7, 2024 15:37
@thingineeer thingineeer added the UX label Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Setting 프로젝트 세팅, 라이브러리 설치 UX 명진😼

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants