fix(notify): restore notification text from PR #223 while keeping icon fix from PR #227 - #232
Conversation
…e keeping icon fix from PR ActivityWatch#227 PR ActivityWatch#227 accidentally reverted the user-friendly text changes from PR ActivityWatch#223: - 'ActivityWatch Server' -> 'ActivityWatch' - 'Server and sync running in background' -> 'Running in the background' The icon fix from ActivityWatch#227 (using ic_stat_notification) is preserved. Fixes ActivityWatch#226
Greptile SummaryRestores the concise foreground-service notification title and description while preserving the dedicated status-bar icon.
Confidence Score: 5/5The PR appears safe to merge, with no actionable correctness, security, or quality issues identified. The change is limited to two static notification strings and preserves the existing foreground-service construction, pending intent, channel, and valid status-bar icon. Important Files Changed
Reviews (1): Last reviewed commit: "fix(notify): restore notification text f..." | Re-trigger Greptile |
|
CI-green and mergeable (Greptile 5/5) — waiting only on a maintainer click. This PR is ready to merge, but the bot has pull-only access to this repo and can't self-merge — surfacing it here so it isn't lost. The monitoring loop will stop re-flagging it now that this note is posted. |
Problem
PR #227 accidentally reverted user-friendly text changes from PR #223 while adding the notification icon fix.
Notification text was changed from the clean version:
But the icon fix (using ic_stat_notification) should be preserved.
Solution
Restore the notification text improvements from PR #223 while keeping the icon fix from PR #227.
The final notification now has:
Related
Fixes #226