Page MenuHomePhabricator
Differential D11914

[native] Consume `useAvailableThreadMemberActions` in `native/ThreadSettingsMember`
ClosedPublic
Actions

Authored by atul on Mon, May 6, 1:13 PM.
Tags
None
Referenced Files
F1783154: D11914.id39872.diff
Sat, May 18, 9:58 AM
Unknown Object (File)
Fri, May 17, 10:43 AM
Unknown Object (File)
Sun, May 12, 12:27 PM
Unknown Object (File)
Sat, May 11, 11:12 AM
Unknown Object (File)
Thu, May 9, 8:33 PM
Unknown Object (File)
Mon, May 6, 3:28 PM
Unknown Object (File)
Mon, May 6, 3:24 PM
Unknown Object (File)
Mon, May 6, 3:15 PM
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
varun
will
Commits
rCOMMdf13d56f8fc9: [native] Consume `useAvailableThreadMemberActions` in…
Summary

Similar to D11913, but for ThreadSettingsMember.

This was last usage of getAvailbleThreadMemberAction, so we also go ahead and remove it.

Depends on D11913

Test Plan

flow + close reading for now

Logged value of getAvailbleThreadMemberAction and useAvailableThreadMemberActions and ensured value was the same. Assuming the logic in ThreadSettingsMember that works with value is and remains correct.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
shared/
37 lines
native/
chat/
settings/
24 lines

Diff 40177

View Options

lib/shared/thread-utils.js

Loading...
View Options

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

Loading...