Skip to content

refactor: added notification groups#58

Merged
ardevd merged 2 commits into
mainfrom
notification-groups
Apr 14, 2026
Merged

refactor: added notification groups#58
ardevd merged 2 commits into
mainfrom
notification-groups

Conversation

@ardevd

@ardevd ardevd commented Apr 14, 2026

Copy link
Copy Markdown
Owner

No description provided.

@ardevd ardevd self-assigned this Apr 14, 2026
Copilot AI review requested due to automatic review settings April 14, 2026 19:27
@ardevd ardevd added the enhancement New feature or request label Apr 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces Android notification channel groups so related notifications (background workers vs timers) are organized together in system notification settings.

Changes:

  • Added new string resources for notification group names.
  • Created NotificationChannelGroups and assigned existing notification channels to those groups for background workers.
  • Created a timers notification group and assigned the active timer notification channel to it.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
app/src/main/res/values/strings.xml Adds default (en) strings for the new notification groups.
app/src/main/res/values-no/strings.xml Adds Norwegian string for the background notification group.
app/src/main/java/net/ardevd/tagius/features/background/ZombieCheckWorker.kt Creates/uses the “background tasks” notification group for zombie alerts.
app/src/main/java/net/ardevd/tagius/features/background/WeeklySummaryWorker.kt Creates/uses the “background tasks” notification group for weekly summaries.
app/src/main/java/net/ardevd/tagius/features/background/TimerNotificationManager.kt Creates/uses the “timers” notification group for the active timer channel.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/src/main/res/values-no/strings.xml
@ardevd ardevd merged commit 8487614 into main Apr 14, 2026
2 checks passed
@ardevd ardevd deleted the notification-groups branch April 14, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants