feat: add realtime notification bridge #63

Closed
sealad886 wants to merge 8 commits from sealad886/notifications-core into main AGit
Contributor

Summary

  • add Tauri notification plugin wiring and capabilities for native desktop/mobile notification display
  • add Grindr notification API helpers for FCM token registration and notification acknowledgement
  • add a realtime notification bridge for WebSocket notification events, including native display formatting and ack-after-display behavior
  • cover the new API helpers, native notification formatting/permission flow, and realtime bridge behavior with unit tests

Stack

Depends on #59, #60, #61, and #62.

Validation

  • bun run test
  • bun run build
  • git diff --check

Note: bun run check still reports existing origin/main Svelte type errors in NavBar.svelte route IDs and ImageCarousel.svelte; this stack does not resolve those pre-existing errors.

## Summary - add Tauri notification plugin wiring and capabilities for native desktop/mobile notification display - add Grindr notification API helpers for FCM token registration and notification acknowledgement - add a realtime notification bridge for WebSocket notification events, including native display formatting and ack-after-display behavior - cover the new API helpers, native notification formatting/permission flow, and realtime bridge behavior with unit tests ## Stack Depends on #59, #60, #61, and #62. ## Validation - `bun run test` - `bun run build` - `git diff --check` Note: `bun run check` still reports existing `origin/main` Svelte type errors in `NavBar.svelte` route IDs and `ImageCarousel.svelte`; this stack does not resolve those pre-existing errors.
Owner

Pending changes in #59 and #60 before reviewing

Pending changes in #59 and #60 before reviewing
Author
Contributor

Base blockers are updated now: #59 was addressed in 11feccb, and #60 was addressed in 85ba445. Ready for review.

Base blockers are updated now: #59 was addressed in `11feccb`, and #60 was addressed in `85ba445`. Ready for review.
sealad886 force-pushed sealad886/notifications-core from 6684143092 to 9ea81cd09c 2026-05-12 23:11:46 +02:00 Compare
sealad886 force-pushed sealad886/notifications-core from 9ea81cd09c to 73c8f297d8 2026-05-13 00:10:14 +02:00 Compare
Author
Contributor

Updated stack after the pending base/review comments. Rebased onto current origin/main, replayed over #60 fixes (AppCache profile cache storage, Tauri 2.11 Rust/JS alignment), and force-pushed the refreshed AGit ref. This PR now has clean git merge-tree against origin/main and passes: git diff --check, bun run test:unit, bun run test:rust, bun run build. Ready for review.

Updated stack after the pending base/review comments. Rebased onto current `origin/main`, replayed over #60 fixes (AppCache profile cache storage, Tauri 2.11 Rust/JS alignment), and force-pushed the refreshed AGit ref. This PR now has clean `git merge-tree` against `origin/main` and passes: `git diff --check`, `bun run test:unit`, `bun run test:rust`, `bun run build`. Ready for review.
Owner

Notifications are tracked separately as a background-delivery problem rather than a WS-listener problem. Closing since the base has moved too far. Thank you.

Notifications are tracked separately as a background-delivery problem rather than a WS-listener problem. Closing since the base has moved too far. Thank you.
hloth closed this pull request 2026-08-23 22:15:25 +02:00

Pull request closed

Sign in to join this conversation.
No description provided.