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

	F3517426: D13259.id44093.diff
	Sun, Dec 22, 5:21 PM

	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)
	Nov 22 2024, 4:07 PM

	Unknown Object (File)
	Nov 22 2024, 4:07 PM

	Unknown Object (File)
	Nov 14 2024, 1:01 PM

	Unknown Object (File)
	Nov 14 2024, 12:59 PM

View All 51 Files

Subscribers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #43925)	How about renaming it?
134–138 ↗	(On Diff #43925)	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

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

Path

Size

lib/

hooks/

thread-hooks.js

12 lines

Diff 44093

lib/hooks/thread-hooks.js

Loading...