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

Feat : 알림 권한 설정 #193

Merged
merged 21 commits into from
Mar 25, 2024
Merged

Feat : 알림 권한 설정 #193

merged 21 commits into from
Mar 25, 2024

Conversation

huiwoo-jo
Copy link
Contributor

@huiwoo-jo huiwoo-jo commented Mar 20, 2024

📮 관련 이슈

✍️ 구현 내용

  • FCM을 위한 알림 권한 설정입니다.
  • 스플래시 화면이 처음 켜질 경우 알림 권한을 요청합니다.
  • 온보딩 화면 3단계에서 알림을 킬 경우 알림 권한을 거부한 상태일때, 설정 화면을 띄웁니다.
  • 설정 화면을 띄우는 이유는 아래와 같습니다.
    • 어플에서는 사용자에게 2번을 초과하여 알림 설정을 요청할 수 없습니다.
    • 스위치 on/off 로직을 위하여 설정 화면으로 이동합니다.
  • 수정 필요
    원활한 개발을 위하여 우선 스플래시와 온보딩만 구현을 하였으나 아래와 같은 수정이 필요합니다.
    • 설정 화면으로 이동하기 전 다이어그램을 띄워 사용자에게 전달하도록 수정할 필요가 있습니다.
    • 어플 세팅 화면에서 알림을 킬 때 똑같은 로직이 필요하며, 수정이 필요합니다.

📷 구현 영상

image

스플래시에서 허용 스플래시에서 거부
https://github.com/TeamDMU/DMU-Android/assets/84004687/bbf49ed0-ed4b-49d6-96d8-9ff97d6f552c https://github.com/TeamDMU/DMU-Android/assets/84004687/a3a37557-085b-488f-8584-794eba070895

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@huiwoo-jo huiwoo-jo added 🤗 FEATURE Develop this project 🩵희우 labels Mar 20, 2024
@huiwoo-jo huiwoo-jo requested review from hoyahozz and m6z1 March 20, 2024 22:51
@huiwoo-jo huiwoo-jo self-assigned this Mar 20, 2024
@huiwoo-jo huiwoo-jo changed the title Feat : 알림 권한 설 Feat : 알림 권한 설정 Mar 20, 2024
Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

코멘트 확인 한 번만 부탁드립니다!!! 고생하셨어용

Copy link
Contributor Author

@huiwoo-jo huiwoo-jo left a comment

Choose a reason for hiding this comment

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

코멘트 남겨주셔서 감사합니다
권한 요청 코드 부분에 대하여 확인해주시면 감사하겠습니다

@huiwoo-jo huiwoo-jo requested a review from m6z1 March 23, 2024 22:40
Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

굿굿 tv 👍

@huiwoo-jo huiwoo-jo merged commit 6c29fc8 into develop Mar 25, 2024
1 check passed
@huiwoo-jo huiwoo-jo deleted the feature/permission-alarm branch March 25, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤗 FEATURE Develop this project 🩵희우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

알림 권한 설정
2 participants