Skip to content

feat/mantra-chant-fcm#483

Merged
tentamdin merged 17 commits into
developfrom
feat/mantra-navigation
Jul 3, 2026
Merged

feat/mantra-chant-fcm#483
tentamdin merged 17 commits into
developfrom
feat/mantra-navigation

Conversation

@CodingWithTashi

@CodingWithTashi CodingWithTashi commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator
  • Chant local push notifications
  • mantra local push notification
  • fcm setup
  • Fixed local push notification issue for 3c (now removed same-day notification if time is passed)
  • Other fixes

@CodingWithTashi CodingWithTashi changed the title Feat/mantra navigation feat/mantra-chant-fcm Jul 2, 2026
@CodingWithTashi CodingWithTashi marked this pull request as ready for review July 2, 2026 02:57
@greptile-apps

greptile-apps Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Safe to merge — changes are well-scoped to the notification subsystem with thorough test coverage for the new accumulator/timer paths and the updated case 3b behaviour.

No logic defects were found. The new ID ranges are correctly bounded, toggle state management follows the established pattern, and the navigation fix is properly guarded with a one-shot flag and a mounted check. Only minor documentation inconsistencies were identified.

No files require special attention.

Reviews (5): Last reviewed commit: "fix:audio" | Re-trigger Greptile

Comment thread lib/features/push_notifications/application/push_notification_service.dart Outdated
Comment thread lib/features/push_notifications/presentation/push_message_navigator.dart Outdated
CodingWithTashi and others added 3 commits July 2, 2026 10:36
…service.dart

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
…gator.dart

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@greptile-apps

greptile-apps Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Want your agent to iterate on Greptile's feedback? Try greploops.

@tentamdin tentamdin merged commit b6c850f into develop Jul 3, 2026
1 check failed
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