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

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

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Apr 26 2024, 7:58 PM

Harbormaster completed remote builds in B28545: Diff 39601.Apr 26 2024, 8:36 PM

atul requested review of this revision.Apr 26 2024, 8:36 PM

ashoat accepted this revision.Apr 28 2024, 1:47 AM

This revision is now accepted and ready to land.Apr 28 2024, 1:47 AM

land

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

Closed by commit rCOMM2d4677a72526: [lib] Narrow `canToggleMessagePin` and `useCanToggleMessagePin` types. · 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.May 3 2024, 8:07 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

message-pinning-utils.js

6 lines

Diff 39764

View Options

lib/utils/message-pinning-utils.js

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