Page MenuHomePhabricator
Differential D11791

[native] Replace `threadHasPermission` with `useThreadHasPermission` in `ThreadSettings`
ClosedPublic
Actions

Authored by atul on Thu, Apr 25, 2:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 6, 12:26 PM
Unknown Object (File)
Thu, May 2, 11:20 PM
Unknown Object (File)
Wed, May 1, 1:30 PM
Unknown Object (File)
Wed, May 1, 12:02 PM
Unknown Object (File)
Tue, Apr 30, 8:15 PM
Unknown Object (File)
Tue, Apr 30, 7:22 PM
Unknown Object (File)
Mon, Apr 29, 10:40 AM
Unknown Object (File)
Mon, Apr 29, 9:13 AM
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
Commits
rCOMM136bd75e5a11: [native] Replace `threadHasPermission` with `useThreadHasPermission` in…
Summary

Similar to D11790, but for ThreadSettings

Depends on D11790

Test Plan

flow + close reading

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Thu, Apr 25, 2:00 PM
atul added a child revision: D11792: [lib] Turn `canPromoteSidebar` to `useCanPromoteSidebar` hook.Thu, Apr 25, 2:03 PM
Harbormaster completed remote builds in B28487: Diff 39517.Thu, Apr 25, 2:26 PM
atul requested review of this revision.Thu, Apr 25, 2:26 PM
ashoat accepted this revision.Thu, Apr 25, 2:29 PM
This revision is now accepted and ready to land.Thu, Apr 25, 2:29 PM
atul updated this revision to Diff 39575.Fri, Apr 26, 11:43 AM

land

This revision was landed with ongoing or failed builds.Fri, Apr 26, 11:46 AM
Closed by commit rCOMM136bd75e5a11: [native] Replace `threadHasPermission` with `useThreadHasPermission` in… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM136bd75e5a11: [native] Replace `threadHasPermission` with `useThreadHasPermission` in….
Harbormaster completed remote builds in B28528: Diff 39575.Fri, Apr 26, 12:11 PM
atul mentioned this in D11816: [lib] Replace `threadHasPermission` with `useThreadHasPermission` in `useCanEditMessage`.Fri, Apr 26, 12:49 PM
atul mentioned this in rCOMMbf7178f18877: [lib] Replace `threadHasPermission` with `useThreadHasPermission` in….Thu, May 2, 9:59 AM

Revision Contents
Changeset List

PathSize
native/
chat/
settings/
107 lines

Diff 39582

View Options

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

Loading...