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.

Sign up for GitHub

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

Jump to bottom

[2.0.1] 앱 최초 설치자의 경우 앱이 스플래시 화면에서 멈추는 이슈 및 피드백 이슈 수정 #208

Merged
merged 4 commits into from
May 19, 2024
Merged

[2.0.1] 앱 최초 설치자의 경우 앱이 스플래시 화면에서 멈추는 이슈 및 피드백 이슈 수정 #208

merged 4 commits into from
May 19, 2024

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented May 18, 2024 •
edited
Loading

내용

  • 앱을 처음 설치한 유저의 경우 화면이 스플래시 단계에서 멈추는 크리티컬한 버그 수정

    • 원인: fcmToken이 없는 경우 onChange에서 관리하는데 최초 설치자의 경우 로컬 디바이스에 설치된 fcmToke이 "" 상태이고, oldValue.isEmpty면 return 하는 부분에 의해 더이상 로직이 진행되지 않는 문제.
    • oldValue가 empty여도 로직이 진행되도록 수정

  • 피드백 405 에러 이슈 수정

    • http로 post 요청시 리다이렉트 되며, 이때 메소드가 get으로 전환되었음.
    • 해결방법: https로 적용

@lgvv lgvv self-assigned this May 18, 2024
@lgvv lgvv requested review from sunshiningsoo and wonniiii May 18, 2024 15:58
@lgvv lgvv added 버그 쿠링 쪽 버그 더보기 더보기 기능 (피드백 포함) labels May 18, 2024
@lgvv
Copy link
Member Author

lgvv commented May 18, 2024 •
edited
Loading

/쿠링 패키지 빌드

@github-actions GitHub Actions
Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

@lgvv lgvv changed the title [2.0.1] 앱 최초 설치자 앱 런칭이 안되는 이슈 및 피드백 이슈 수정 [2.0.1] 앱 최초 설치자의 경우 앱이 스플래시 화면에서 멈추는 이슈 및 피드백 이슈 수정 May 18, 2024
@github-actions GitHub Actions
Copy link

✅ 패키지를 성공적으로 빌드했습니다.

sunshiningsoo
sunshiningsoo approved these changes May 19, 2024
View reviewed changes
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

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

🫡

@lgvv
Copy link
Member Author

lgvv commented May 19, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label May 19, 2024
@github-actions github-actions bot merged commit 19480d5 into develop/2.0.0 May 19, 2024
@github-actions GitHub Actions
Copy link

🕊️ PR #208이 머지되었습니다.

@github-actions github-actions bot deleted the feature/lgvv/user-token branch May 19, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunshiningsoo sunshiningsoo sunshiningsoo approved these changes

@wonniiii wonniiii Awaiting requested review from wonniiii

Assignees

@lgvv lgvv

Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 더보기 더보기 기능 (피드백 포함) 버그 쿠링 쪽 버그
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lgvv @sunshiningsoo