Page MenuHomePhabricator
Differential D11754

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

Authored by atul on Tue, Apr 23, 2:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 1, 1:37 PM
Unknown Object (File)
Tue, Apr 30, 12:13 PM
Unknown Object (File)
Mon, Apr 29, 12:00 PM
Unknown Object (File)
Sat, Apr 27, 9:19 AM
Unknown Object (File)
Tue, Apr 23, 8:58 PM
Unknown Object (File)
Tue, Apr 23, 8:57 PM
Unknown Object (File)
Tue, Apr 23, 8:55 PM
Unknown Object (File)
Tue, Apr 23, 3:34 PM
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
varun
will
Commits
rCOMM5c7f9868ce1a: [native] Replace `threadHasPermission` with `useThreadHasPermission` in…
Summary

Similar to D11753, but for ThreadSettings

Depends on D11753

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
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 23, 2:31 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 23, 2:31 PM
Harbormaster completed remote builds in B28426: Diff 39431.Tue, Apr 23, 3:14 PM
atul requested review of this revision.Tue, Apr 23, 3:14 PM
tomek accepted this revision.Wed, Apr 24, 4:53 AM

Will test generally at end to make sure things work as expected.

Is this something that will be done later in the stack?

This revision is now accepted and ready to land.Wed, Apr 24, 4:53 AM
atul added a comment.Thu, Apr 25, 10:13 AM
In D11754#337417, @tomek wrote:

Will test generally at end to make sure things work as expected.

Is this something that will be done later in the stack?

Yeah and have been doing basic sanity checks throughout with log statements

atul added a child revision: D11786: [native] Replace `threadHasPermission` with `useThreadHasPermission` in `ViewInviteLinksScreen`.Thu, Apr 25, 1:54 PM
atul updated this revision to Diff 39561.Fri, Apr 26, 10:55 AM

land

Harbormaster completed remote builds in B28516: Diff 39561.Fri, Apr 26, 11:14 AM
Closed by commit rCOMM5c7f9868ce1a: [native] Replace `threadHasPermission` with `useThreadHasPermission` in… (authored by atul). · Explain WhyFri, Apr 26, 11:46 AM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM5c7f9868ce1a: [native] Replace `threadHasPermission` with `useThreadHasPermission` in….

Revision Contents
Changeset List

PathSize
native/
chat/
settings/
16 lines

Diff 39576

View Options

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

Loading...