Page MenuHomePhabricator
Differential D13335

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

Authored by inka on Fri, Sep 13, 6:58 AM.
Tags
None
Referenced Files
F2748993: D13335.id44187.diff
Wed, Sep 18, 10:59 AM
F2744190: D13335.id44174.diff
Wed, Sep 18, 4:23 AM
Unknown Object (File)
Tue, Sep 17, 4:38 AM
Unknown Object (File)
Sun, Sep 15, 11:28 PM
Unknown Object (File)
Fri, Sep 13, 8:44 AM
Unknown Object (File)
Fri, Sep 13, 8:44 AM
Unknown Object (File)
Fri, Sep 13, 8:43 AM
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.Fri, Sep 13, 6:58 AM
Herald added a subscriber: ashoat. · View Herald TranscriptFri, Sep 13, 6:58 AM
inka added a parent revision: D13333: [lib] update threadLabel to handle thick threads.Fri, Sep 13, 6:58 AM
inka added a child revision: D13320: [web][native] Update UI to not display GENESIS over thick threads.Fri, Sep 13, 7:19 AM
Harbormaster completed remote builds in B31614: Diff 44174.Fri, Sep 13, 7:19 AM
inka requested review of this revision.Fri, Sep 13, 7:19 AM
tomek accepted this revision.Fri, Sep 13, 7:24 AM
This revision is now accepted and ready to land.Fri, Sep 13, 7:24 AM
tomek added inline comments.Fri, Sep 13, 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 WhyFri, Sep 13, 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.Fri, Sep 13, 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...