Page MenuHomePhabricator
Differential D13335

[lib][web][native] Refactor usingOlmViaTunnelbrokerForDMs
ClosedPublic
Actions

Authored by inka on Sep 13 2024, 6:58 AM.
Tags
None
Referenced Files
F3233114: D13335.id44174.diff
Tue, Nov 12, 2:31 PM
Unknown Object (File)
Sun, Nov 10, 3:12 AM
Unknown Object (File)
Mon, Oct 14, 2:29 PM
Unknown Object (File)
Mon, Oct 14, 2:29 PM
Unknown Object (File)
Mon, Oct 14, 2:29 PM
Unknown Object (File)
Mon, Oct 14, 2:28 PM
Unknown Object (File)
Oct 1 2024, 3:08 AM
Unknown Object (File)
Sep 28 2024, 2:22 PM
View All 20 Files
Subscribers
ashoat

Details

Reviewers
tomek
kamil
Commits
rCOMMaed77d4733bd: [lib][web][native] Refactor usingOlmViaTunnelbrokerForDMs
Summary

issue: ENG-9263
We want to allow thick threads for dev and staff

Test Plan

flow, tested that creating thick threads works on native

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Sep 13 2024, 6:58 AM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 13 2024, 6:58 AM
inka added a parent revision: D13333: [lib] update threadLabel to handle thick threads.Sep 13 2024, 6:58 AM
inka added a child revision: D13320: [web][native] Update UI to not display GENESIS over thick threads.Sep 13 2024, 7:19 AM
Harbormaster completed remote builds in B31614: Diff 44174.Sep 13 2024, 7:19 AM
inka requested review of this revision.Sep 13 2024, 7:19 AM
tomek accepted this revision.Sep 13 2024, 7:24 AM
This revision is now accepted and ready to land.Sep 13 2024, 7:24 AM
tomek added inline comments.Sep 13 2024, 8:06 AM
lib/hooks/flag-hooks.js
9–12 ↗(On Diff #44174)

Maybe it's better to use useStaffCanSee.

Closed by commit rCOMMaed77d4733bd: [lib][web][native] Refactor usingOlmViaTunnelbrokerForDMs (authored by inka). · Explain WhySep 13 2024, 8:59 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMMaed77d4733bd: [lib][web][native] Refactor usingOlmViaTunnelbrokerForDMs.
inka added inline comments.Sep 13 2024, 9:02 AM
lib/hooks/flag-hooks.js
9–12 ↗(On Diff #44174)

That is a native specific flag

Revision Contents
Changeset List

Diff 44186

View Options

lib/hooks/flag-hooks.js

Loading...
View Options

lib/shared/thread-actions-utils.js

Loading...
View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/utils/services-utils.js

Loading...
View Options

native/chat/chat-thread-list.react.js

Loading...
View Options

native/chat/compose-thread-button.react.js

Loading...
View Options

native/chat/message-list-container.react.js

Loading...
View Options

native/input/input-state-container.react.js

Loading...
View Options

web/chat/thread-list-provider.js

Loading...
View Options

web/input/input-state-container.react.js

Loading...
View Options

web/utils/thread-utils.js

Loading...