[web] Replace `threadHasPermission` with `useThreadHasPermission` in `MembersModal`
ClosedPublic
Actions

Authored by atul on Tue, Apr 23, 1:37 PM.

Details

Reviewers

varun
ginsu
tomek
will

Commits

rCOMMb6c3358bdf37: [web] Replace `threadHasPermission` with `useThreadHasPermission` in…

Summary

Similar to D11751, but for MembersModal.

Depends on D11751

Test Plan

flow + close reading.

Will test generally at end to make sure things work as expected. Don't think it's practical to manually test each of these trivial changes.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Tue, Apr 23, 1:37 PM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 23, 1:37 PM

atul added a child revision: D11753: [web] Replace `threadHasPermission` with `useThreadHasPermission` in `ThreadSettings*`.Tue, Apr 23, 1:49 PM

Harbormaster completed remote builds in B28424: Diff 39429.Tue, Apr 23, 1:53 PM

atul requested review of this revision.Tue, Apr 23, 1:53 PM

tomek accepted this revision.Wed, Apr 24, 4:44 AM

This revision is now accepted and ready to land.Wed, Apr 24, 4:44 AM

rebase and land

Harbormaster completed remote builds in B28455: Diff 39466.Wed, Apr 24, 2:16 PM

Closed by commit rCOMMb6c3358bdf37: [web] Replace `threadHasPermission` with `useThreadHasPermission` in… (authored by atul). · Explain WhyWed, Apr 24, 6:58 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMb6c3358bdf37: [web] Replace `threadHasPermission` with `useThreadHasPermission` in….

Revision Contents
Changeset List

Path

Size

web/

modals/

threads/

members/

members-modal.react.js

4 lines

Diff 39466

View Options

web/modals/threads/members/members-modal.react.js

[web] Replace `threadHasPermission` with `useThreadHasPermission` in `MembersModal`ClosedPublicActions