Skip to content

feat: Desktop notification setting #391

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
charlesvien merged 2 commits into from
Jan 6, 2026
Merged

feat: Desktop notification setting #391

charlesvien merged 2 commits into from
Jan 6, 2026

Conversation

@charlesvien
Copy link
Contributor

@charlesvien charlesvien commented Jan 5, 2026 •
edited
Loading

  • Adds a setting to enable/disable desktop badge notifications for when the agent has completed a chat task (default = enabled)

@charlesvien charlesvien mentioned this pull request Jan 5, 2026
Merged
@charlesvien Graphite App
Copy link
Contributor Author

charlesvien commented Jan 5, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Jan 5, 2026
Merged
Merged
Closed
Merged
@charlesvien charlesvien changed the title Desktop notification setting feat: Desktop notification setting Jan 5, 2026
@charlesvien charlesvien marked this pull request as ready for review January 5, 2026 23:26
@charlesvien charlesvien requested a review from a team as a code owner January 5, 2026 23:26
This was referenced Jan 6, 2026
Merged
Merged
@charlesvien charlesvien force-pushed the 01-05-desktop_notification_setting branch from c170f21 to b9bd64f Compare January 6, 2026 19:34
@charlesvien charlesvien force-pushed the 01-05-fix_a_bug_where_stale_plans_persisted_forever_in_the_chat branch from 21c813f to 474da6a Compare January 6, 2026 19:34
@charlesvien charlesvien force-pushed the 01-05-desktop_notification_setting branch from b9bd64f to 9da07a6 Compare January 6, 2026 19:37
@charlesvien charlesvien force-pushed the 01-05-fix_a_bug_where_stale_plans_persisted_forever_in_the_chat branch from 474da6a to 31b4757 Compare January 6, 2026 19:37
@charlesvien Graphite App
Copy link
Contributor Author

charlesvien commented Jan 6, 2026 •
edited
Loading

Merge activity

  • Jan 6, 7:42 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 6, 7:53 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 6, 7:55 PM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien changed the base branch from 01-05-fix_a_bug_where_stale_plans_persisted_forever_in_the_chat to graphite-base/391 January 6, 2026 19:48
@charlesvien charlesvien changed the base branch from graphite-base/391 to main January 6, 2026 19:51
@charlesvien charlesvien force-pushed the 01-05-desktop_notification_setting branch from 0a49f69 to 275434a Compare January 6, 2026 19:52
@charlesvien charlesvien merged commit a6003aa into main Jan 6, 2026
11 checks passed
@charlesvien charlesvien deleted the 01-05-desktop_notification_setting branch January 6, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joshsny joshsny joshsny approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@charlesvien @joshsny