rm-hull / github-pr-dashboard / 19479713465

56%

push

github

feat: Add desktop notifications for new PRs (#89)

* feat: Add desktop notifications for new PRs

Adds logic to detect an increase in the number of displayed
pull requests and trigger a native browser notification.

This uses the `Notification` API to alert the user when new PRs
arrive without needing to refresh the page.

* feat: Add toggle for desktop notifications

Adds explicit user control for desktop notifications when new
pull requests are detected.

This feature manages notification permission requests and updates
the existing notification logic to rely on the new setting.

Additionally, notification messages are improved using the
`pluralize` library for grammatical accuracy.

* refactor: Move notification logic to component

This commit extracts all browser notification handling and
permission requesting into a new dedicated component,
`<Notifications />`.

This improves separation of concerns:

*   Removes notification logic from `PullRequestsList`.
*   Removes permission request logic from `SettingsForm`.
*   Centralizes notification management in one place.

* chore: Log notification state for debugging

Adds a `console.log` statement to trace the current `count`, the
previous count stored in `prevCountRef.current`, and the current
`Notification.permission`.

Also updates the `useEffect` dependency array in `Notifications.tsx`.

* feat: Add public notification.png

This asset is required to correctly display the official
notification icon in the application UI.

* perf: Use WEBP for notification icon assets

Replaces the `.png` notification icon with the more efficient
`.webp` format for improved performance.

- Updates the notification component to use the new icon path.
- Adds `requireInteraction` to desktop notifications in
  non-production environments for easier testing and review.
- Includes minor code refactoring and import cleanup.

* Update src/components/settings/SettingsForm.tsx

Co-authored-by: gemini-code-assist[bot] <1769615... (continued)

9 of 11 branches covered (81.82%)

Branch coverage included in aggregate %.

27 of 54 relevant lines covered (50.0%)

2.19 hits per line