feat: add realtime notification bridge #63
No reviewers
Labels
No labels
app
bug
client side exclusive
🗳️ community vote
⚠️ critical
discussion
docs
duplicate
emulator
enhancement
help wanted
invalid
not open grind
question
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
open-grind/open-grind!63
Loading…
Reference in a new issue
No description provided.
Delete branch "sealad886/notifications-core"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
Stack
Depends on #59, #60, #61, and #62.
Validation
bun run testbun run buildgit diff --checkNote:
bun run checkstill reports existingorigin/mainSvelte type errors inNavBar.svelteroute IDs andImageCarousel.svelte; this stack does not resolve those pre-existing errors.Pending changes in #59 and #60 before reviewing
Base blockers are updated now: #59 was addressed in
11feccb, and #60 was addressed in85ba445. Ready for review.6684143092to9ea81cd09c9ea81cd09cto73c8f297d8Updated 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 cleangit merge-treeagainstorigin/mainand passes:git diff --check,bun run test:unit,bun run test:rust,bun run build. Ready for review.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.
Pull request closed