Page MenuHomePhabricator
Differential D13449

[web][native] fix relationship buttons in thread settings
ClosedPublic
Actions

Authored by kamil on Sep 24 2024, 7:02 AM.
Tags
None
Referenced Files
F3299756: D13449.id44538.diff
Sun, Nov 17, 3:22 PM
F3299749: D13449.id44529.diff
Sun, Nov 17, 3:17 PM
Unknown Object (File)
Fri, Nov 1, 2:14 PM
Unknown Object (File)
Fri, Nov 1, 2:14 PM
Unknown Object (File)
Fri, Nov 1, 2:14 PM
Unknown Object (File)
Sun, Oct 20, 10:26 PM
Unknown Object (File)
Oct 17 2024, 3:32 PM
Unknown Object (File)
Oct 17 2024, 3:32 PM
View All 31 Files
Subscribers
None

Details

Reviewers
tomek
ashoat
Commits
rCOMMd788f39fa013: [web][native] fix relationship buttons in thread settings
Summary

ENG-9320.

  1. Fixing rendering relationship tab on web condition.
  2. Updating check on native to support thick threads.
Test Plan

Buttons visible only on personal threads.

Diff Detail

Repository
rCOMM Comm
Branch
fixes-3
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Sep 24 2024, 7:02 AM
kamil held this revision as a draft.
kamil published this revision for review.Sep 24 2024, 7:02 AM
tomek accepted this revision.Sep 24 2024, 7:13 AM
tomek added inline comments.
web/modals/threads/settings/thread-settings-modal.react.js
105
This revision is now accepted and ready to land.Sep 24 2024, 7:13 AM
Harbormaster completed remote builds in B31817: Diff 44485.Sep 24 2024, 7:20 AM
kamil added inline comments.Sep 25 2024, 5:37 AM
web/modals/threads/settings/thread-settings-modal.react.js
105

we need entire threadInfo

kamil updated this revision to Diff 44529.Sep 25 2024, 5:40 AM

address review

Harbormaster completed remote builds in B31841: Diff 44529.Sep 25 2024, 5:58 AM
Closed by commit rCOMMd788f39fa013: [web][native] fix relationship buttons in thread settings (authored by kamil). · Explain WhySep 25 2024, 6:00 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMd788f39fa013: [web][native] fix relationship buttons in thread settings.

Revision Contents
Changeset List

PathSize
lib/
types/
9 lines
native/
chat/
settings/
6 lines
web/
modals/
threads/
settings/
13 lines

Diff 44485

View Options

lib/types/thread-types-enum.js

Loading...
View Options

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

Loading...
View Options

web/modals/threads/settings/thread-settings-modal.react.js

Loading...