Skip to content

222 알림 권한 관리#270

Merged
SeoHyeonSim merged 15 commits into
mainfrom
222-알림-권한-관리
Jun 28, 2025

Hidden character warning

The head ref may contain hidden characters: "222-\uc54c\ub9bc-\uad8c\ud55c-\uad00\ub9ac"
Merged

222 알림 권한 관리#270
SeoHyeonSim merged 15 commits into
mainfrom
222-알림-권한-관리

Conversation

@jjoonleo

Copy link
Copy Markdown
Contributor

Describe your changes

  • 알림 허용 요청 화면 생성
  • 알림 권한 획득 및 서버에 전달 로직 개발

Issue ticket number and link

#226 #222

@jjoonleo jjoonleo added this to the 푸시 알림 milestone Jun 22, 2025
@jjoonleo jjoonleo requested a review from SeoHyeonSim June 22, 2025 07:04
@jjoonleo jjoonleo self-assigned this Jun 22, 2025
@jjoonleo jjoonleo linked an issue Jun 22, 2025 that may be closed by this pull request
@github-actions

github-actions Bot commented Jun 22, 2025

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 7bdbbb5):

https://ontime-c63f1--pr270-222-66w1e8c8.web.app

(expires Sun, 29 Jun 2025 07:33:48 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7d60a2eae4c4a5839220ed6a57552a3b61cce5a3

@github-actions

github-actions Bot commented Jun 22, 2025

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 7bdbbb5):

https://on-time-front-widgetbook--pr270-222-f3jm06se.web.app

(expires Sun, 29 Jun 2025 07:34:22 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e53128a376fed0209d449279de9c0d94c83cdd8b

jjoonleo added 8 commits June 22, 2025 16:30
…on screen

- Added a new NotificationAllowScreen to request notification permissions from users.
- Updated NotificationService to handle permission requests for web and mobile platforms.
- Modified the main app flow to check for notification permissions and redirect to the allow screen if not granted.
- Removed direct notification token request from the bottom navigation bar.
- Enhanced the theme with a custom FilledButton style.
- Updated Firebase messaging service worker to properly handle push notifications.
- Added new assets including SVG images for the notification allow screen.
@SeoHyeonSim SeoHyeonSim merged commit 518baaf into main Jun 28, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

알림 권한 관리

2 participants