Skip to content

[codex] Fix alarm and notification permission flows - #508

Merged
jjoonleo merged 4 commits into
mainfrom
codexd/alarm-permission-flow
May 11, 2026
Merged

jjoonleo merged 4 commits into
mainfrom
codexd/alarm-permission-flow

Conversation

@jjoonleo

Copy link
Copy Markdown
Contributor

Summary

  • add a first-use alarm permission gate after the notification gate
  • keep alarm settings in sync with native alarm permission, including My Page recovery
  • simplify first-use notification permission so it does not open Settings
  • add a My Page notification permission switch that reflects OS permission state

Closes #425.

Validation

  • flutter analyze
  • flutter test test/presentation/alarm_allow/alarm_allow_screen_test.dart test/presentation/my_page/my_page_screen_test.dart
  • flutter test test/presentation/notification_allow/notification_allow_screen_test.dart
  • flutter test test/presentation/my_page/my_page_screen_test.dart test/presentation/notification_allow/notification_allow_screen_test.dart

@jjoonleo
jjoonleo marked this pull request as ready for review May 11, 2026 09:37
@jjoonleo
jjoonleo merged commit 1af5ec3 into main May 11, 2026
3 of 6 checks passed

This branch had an error being deployed

1 failed deployment
debug — bba7d499 Deployed May 11, 2026 by jjoonleo via build_and_preview #412
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.

알람 디폴트 허용 로직

1 participant