[lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` types
ClosedPublic
Actions

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

Details

Reviewers

ashoat
ginsu
tomek
varun

Commits

rCOMM2d4677a72526: [lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` types

Summary

To make it clear which should be used in which scenario (RawThreadInfo vs ThreadInfo).

Depends on D11820

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:58 PM

Harbormaster completed remote builds in B28545: Diff 39601.Fri, Apr 26, 1:36 PM

atul requested review of this revision.Fri, Apr 26, 1:36 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 rCOMM2d4677a72526: [lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` types (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM2d4677a72526: [lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` types.

Harbormaster completed remote builds in B28645: Diff 39764.Fri, May 3, 1:07 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

message-pinning-utils.js

6 lines

Diff 39768

View Options

lib/utils/message-pinning-utils.js

[lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` typesClosedPublicActions