Paths

Table of Contentst

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

Authored by atul on Apr 26 2024, 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

No Lint Coverage

Unit

No Test Coverage

Event Timeline

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

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

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

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

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

This revision is now accepted and ready to land.Apr 27 2024, 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 39762

View Options

lib/utils/message-pinning-utils.js