Skip to content

222 알림 권한 관리 #1025

222 알림 권한 관리

222 알림 권한 관리 #1025

Triggered via pull request June 22, 2025 07:04
Status Failure
Total duration 19s
Artifacts

commit_lint.yml

on: pull_request
commitlint
15s
commitlint
Fit to window
Zoom out
Zoom in

Annotations

1 error
commitlint
You have commit messages with errors ⧗ input: feat: Implement notification permission handling and allow notification 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. ✖ body's lines must not be longer than 100 characters [body-max-line-length] ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: feat: Add bell-ringing SVG asset for notification feature ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: fix: Enable notification display on push event ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: fix: Remove redundant notification service initialization in main function ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: feat: Refactor notification permission handling and improve initialization logic ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: fix: Remove notification token request from schedule creation button ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: feat: Add NotificationAllowScreen for managing notification permissions ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: feat: Add notification permission check and route for NotificationAllowScreen ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint