We don't support removing members and maintaining this code doesn't make sense.
https://linear.app/comm/issue/ENG-9823/fix-sending-notifs-about-leaving-a-thick-thread
Depends on D13843
Details
Details
- Reviewers
kamil angelika - Commits
- rCOMM58824ef48b2e: [lib] Delete remove members spec
Summary
Test Plan
Flow
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Herald added a subscriber: ashoat. · View Herald TranscriptThu, Oct 31, 8:06 AM2024-10-31 08:06:23 (UTC-7)
Harbormaster returned this revision to the author for changes because remote builds failed.Thu, Oct 31, 8:20 AM2024-10-31 08:20:54 (UTC-7)
Harbormaster completed remote builds in B32488: Diff 45521.Thu, Oct 31, 9:56 AM2024-10-31 09:56:38 (UTC-7)
This revision is now accepted and ready to land.Thu, Oct 31, 11:03 AM2024-10-31 11:03:37 (UTC-7)
Closed by commit rCOMM58824ef48b2e: [lib] Delete remove members spec (authored by tomek). · Explain WhyFri, Nov 8, 4:29 AM2024-11-08 04:29:45 (UTC-8)
This revision was automatically updated to reflect the committed changes.