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

[Feature/#339] 앱 알림 설정 유무 등록 API 연결 #342

Merged

Conversation

JongHoooon
Copy link
Member

@JongHoooon JongHoooon commented Nov 4, 2024

이슈: #339

작업 내용

  • 스플래쉬 화면에서 앱 알림관련 변경 사항이 있는 경우, 업로드한 적이 없는 경우 서버에 알림 설정 값 업로드

고민 사항

  • 처음에 alertAllowState 키값 기반으로 User Defaults에 저장한 값을 사용하려고 했는데, fetchPushNotificationAllowStatus 메소드 반환값이 optional이 아니라 업로드한 적이 없는 경우 대응하기 힘들어 remotelyUploadedPushNotificationAllowStatus 키값을 기반으로 User Defaults 에 서버에 업로드한 값을 저장하고 스플래쉬 화면에서 변경사항이 생겼을 때 체크(isNeedUpdatePushNotificationRemotely use case 활용)해 업로드 하도록 수정.

@JongHoooon JongHoooon self-assigned this Nov 4, 2024
@JongHoooon JongHoooon linked an issue Nov 4, 2024 that may be closed by this pull request
3 tasks
@JongHoooon JongHoooon force-pushed the feature/#339-앱-알림-설정-유뮤-등록-api-연결 branch 2 times, most recently from f6eb7ff to d69127e Compare November 5, 2024 10:18
Copy link
Member

@leemhyungyu leemhyungyu left a comment

Choose a reason for hiding this comment

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

굳!

@JongHoooon JongHoooon force-pushed the feature/#339-앱-알림-설정-유뮤-등록-api-연결 branch from d69127e to 6f6338c Compare November 8, 2024 09:09
@JongHoooon JongHoooon merged commit e294b2d into develop Nov 8, 2024
1 check passed
@JongHoooon JongHoooon deleted the feature/#339-앱-알림-설정-유뮤-등록-api-연결 branch November 8, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

앱 알림 설정 유뮤 등록 API 연결
2 participants