Skip to content

feat(team-inbox): land notification and collaboration followups - #592

Merged
sudomaggie merged 2 commits into
developfrom
junyu/team-inbox-followups-to-develop
Jul 30, 2026
Merged

feat(team-inbox): land notification and collaboration followups#592
sudomaggie merged 2 commits into
developfrom
junyu/team-inbox-followups-to-develop

Conversation

@beruro

@beruro beruro commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Problem

PR #582 was merged into the already-landed #566 head branch, and PR #583 was then merged into the #582 head branch. Both PRs show as merged, but their notification and Team Inbox changes never reached develop.

Solution

Potential risks

  • the recovery touches 98 files across TypeScript and Rust, so merge conflicts are possible if develop changes the same Team Inbox or notification paths before landing
  • notification delivery and realtime teardown are lifecycle-sensitive; regressions could cause duplicate delivery or retained timers
  • mitigations: exact patch-id match, clean replay on current develop, 88 targeted tests, TypeScript typecheck, and cargo check -p org2 --lib all pass
  • do not merge the old feat(notifications): unify background delivery lifecycle #582/feat(team-inbox): harden collaboration handoffs #583 branches into develop again after this PR lands

@sudomaggie
sudomaggie merged commit 0e04b26 into develop Jul 30, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants