Paths

Table of Contentst

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

Authored by inka on Sep 13 2024, 6:58 AM.

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

Branch

inka/fix_marcin

Lint

No Lint Coverage

Unit

No Test Coverage

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	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	That is a native specific flag

Revision Contents
Changeset List

Path

Size

lib/

hooks/

flag-hooks.js

14 lines

shared/

thread-actions-utils.js

14 lines

thread-utils.js

52 lines

utils/

services-utils.js

6 lines

native/

chat/

chat-thread-list.react.js

10 lines

compose-thread-button.react.js

5 lines

message-list-container.react.js

7 lines

input/

input-state-container.react.js

5 lines

web/

chat/

thread-list-provider.js

3 lines

input/

input-state-container.react.js

5 lines

utils/

thread-utils.js

4 lines

Diff 44174

View Options

lib/hooks/flag-hooks.js

View Options

lib/shared/thread-actions-utils.js

View Options

lib/shared/thread-utils.js

View Options

lib/utils/services-utils.js

View Options

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

View Options

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

View Options

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

View Options

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

View Options

web/chat/thread-list-provider.js

View Options

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

View Options

[lib][web][native] Refactor usingOlmViaTunnelbrokerForDMsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 44174

lib/hooks/flag-hooks.js

lib/shared/thread-actions-utils.js

lib/shared/thread-utils.js

lib/utils/services-utils.js

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

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

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

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

web/chat/thread-list-provider.js

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

web/utils/thread-utils.js

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

Revision Contents
Changeset List