Page MenuHomePhabricator
Differential D13259

[lib] Fix sendnig create_sidebar to peers
ClosedPublic
Actions

Authored by inka on Sep 6 2024, 6:52 AM.
Tags
None
Referenced Files
F3516168: D13259.diff
Sun, Dec 22, 12:36 PM
Unknown Object (File)
Mon, Dec 16, 5:10 AM
Unknown Object (File)
Thu, Dec 12, 1:37 AM
Unknown Object (File)
Fri, Nov 22, 4:07 PM
Unknown Object (File)
Fri, Nov 22, 4:07 PM
Unknown Object (File)
Nov 14 2024, 1:01 PM
Unknown Object (File)
Nov 14 2024, 12:59 PM
Unknown Object (File)
Nov 14 2024, 3:02 AM
View All 50 Files
Subscribers
ashoat

Details

Reviewers
tomek
kamil
Commits
rCOMM4a9159903769: [lib] Fix sendnig create_sidebar to peers
Summary

issue: ENG-9134
For sidebars we should also inform the members of the parent about thier creation

Test Plan

Tested that now when a sidebar is created in a thick thread, other members of the thick thread are informed

Diff Detail

Repository
rCOMM Comm
Branch
inka/thread_ops
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Sep 6 2024, 6:52 AM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 6 2024, 6:52 AM
Harbormaster completed remote builds in B31459: Diff 43925.Sep 6 2024, 7:14 AM
inka requested review of this revision.Sep 6 2024, 7:14 AM
inka added a parent revision: D13258: [lib][native] Review usingOlmViaTunnelbrokerForDMs and pendingThreadType.Sep 6 2024, 7:40 AM
inka mentioned this in D13253: [lib][web][native] Update createPendingThread usages.
tomek accepted this revision.Sep 6 2024, 7:43 AM
tomek added inline comments.
lib/hooks/thread-search-hooks.js
114

How about renaming it?

134–138

We shouldn't include the members here - they are already included in the parent thread's members list.

This revision is now accepted and ready to land.Sep 6 2024, 7:43 AM
inka added a child revision: D13266: [lib] Update existingThreadInfoFinder to handle thick threads.Sep 9 2024, 3:09 AM
inka updated this revision to Diff 44084.Sep 12 2024, 7:12 AM

Address review

Harbormaster completed remote builds in B31566: Diff 44084.Sep 12 2024, 7:29 AM
Closed by commit rCOMM4a9159903769: [lib] Fix sendnig create_sidebar to peers (authored by inka). · Explain WhySep 12 2024, 8:04 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMM4a9159903769: [lib] Fix sendnig create_sidebar to peers.

Revision Contents
Changeset List

PathSize
lib/
hooks/
12 lines

Diff 43925

View Options

lib/hooks/thread-search-hooks.js

Loading...