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

강제 업데이트 기능 구현 #125

Merged
merged 4 commits into from
Dec 22, 2023
Merged

Conversation

kimkyunghun3
Copy link
Contributor

개요

  • 강제 업데이트 기능을 구현합니다.

변경사항

  • 스플래쉬 화면에서 앱 버전이 다른 경우 강제 업데이트 팝업을 띄웁니다.

스크린샷

RPReplay_Final1701157160.MP4

@kimkyunghun3 kimkyunghun3 added ✨ Feature New feature or request 🐨 경훈 경훈 담당 ✈️ ASAP 빠른 리뷰 및 머지 부탁드립니당 labels Nov 28, 2023
@kimkyunghun3 kimkyunghun3 self-assigned this Nov 28, 2023
Copy link
Member

@julia0926 julia0926 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!
프로젝트 전반이랑 구현하신 부분이랑 글자 자간이 안맞는데 수정 부탁드려요!


extension SplashInteractor {
func didTapUpdateButton() async {
let appleId = "6455260918"
Copy link
Member

Choose a reason for hiding this comment

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

인터렉터가 appleId 값을 직접 갖고 있어도 괜찮나요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이거 사실 안에 코드보면 URL 에 들어가는 것인데 단순히 url안에 있는 것을 상수로 뺀거 뿐이긴하거든요
요런것도 문제가 되는건가요?(궁금해서..)

Copy link
Member

Choose a reason for hiding this comment

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

아 저도 몰라서 물어봤어요,,ㅎㅎ

@kimkyunghun3
Copy link
Contributor Author

수고하셨습니다~! 프로젝트 전반이랑 구현하신 부분이랑 글자 자간이 안맞는데 수정 부탁드려요!

요거 디자이너분들과 확인해봤는데 잘 맞다네요!

@julia0926
Copy link
Member

julia0926 commented Nov 30, 2023

수고하셨습니다~! 프로젝트 전반이랑 구현하신 부분이랑 글자 자간이 안맞는데 수정 부탁드려요!

요거 디자이너분들과 확인해봤는데 잘 맞다네요!

아 그 디자인말고 코드 스페이스 간격.. 말했던 거였습니다~!
글자 자간이라 헷갈렸을 수도 있겠네요.. 코드 자간이요!

Copy link
Member

@julia0926 julia0926 left a comment

Choose a reason for hiding this comment

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

구우웃!!!

@kimkyunghun3 kimkyunghun3 merged commit 5e00d02 into develop Dec 22, 2023
@kimkyunghun3 kimkyunghun3 deleted the feature/updatePopup branch December 22, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✈️ ASAP 빠른 리뷰 및 머지 부탁드립니당 ✨ Feature New feature or request 🐨 경훈 경훈 담당
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants