Paths

Table of Contentst

[lib] Introduce `useCanToggleMessagePin` and consume in `native/Message`
ClosedPublic
Actions

Authored by atul on Fri, Apr 26, 12:55 PM.

Details

Reviewers

ashoat
ginsu
tomek
varun

Commits

rCOMM30dbaeafde23: [lib] Introduce `useCanToggleMessagePin` and consume in `native/Message`

Summary

Hook version of canToggleMessagePin so we can consume useThreadHasPermission instead of threadHasPermission.

Will leave canToggleMessagePin around since we can still use it on keyserver with RawThreadInfo.

Depends on D11818

Test Plan

flow + close reading

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Fri, Apr 26, 12:55 PM

atul added a child revision: D11820: [web] Replace `canToggleMessagePin` with `useCanToggleMessagePin` in `useMessageTogglePinAction`.

Harbormaster completed remote builds in B28542: Diff 39598.Fri, Apr 26, 1:22 PM

atul requested review of this revision.Fri, Apr 26, 1:22 PM

ashoat accepted this revision.Sat, Apr 27, 6:47 PM

This revision is now accepted and ready to land.Sat, Apr 27, 6:47 PM

land

This revision was landed with ongoing or failed builds.Fri, May 3, 1:01 PM

Closed by commit rCOMM30dbaeafde23: [lib] Introduce `useCanToggleMessagePin` and consume in `native/Message` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM30dbaeafde23: [lib] Introduce `useCanToggleMessagePin` and consume in `native/Message`.

Harbormaster completed remote builds in B28643: Diff 39762.Fri, May 3, 1:09 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

message-pinning-utils.js

20 lines

native/

chat/

message.react.js

8 lines

Diff 39766

View Options

lib/utils/message-pinning-utils.js