[lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`
ClosedPublic
Actions

Authored by atul on Sat, Jun 1, 10:00 PM.

Details

Reviewers

ashoat
ginsu
tomek
varun

Commits

rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`

Summary

This change is kind of useless. Thought I'd be able to do a lot more narrowing of types "along" the memberHasAdminPowers callstack, but this was pretty much it. Figured I may as well put it up.

Depends on D12265

Test Plan

NA, flow

Diff Detail

Repository

rCOMM Comm

Branch

arcpatch-D12266 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Sat, Jun 1, 10:00 PM

Harbormaster completed remote builds in B29343: Diff 40823.Sat, Jun 1, 10:17 PM

atul requested review of this revision.Sat, Jun 1, 10:17 PM

ashoat accepted this revision.Sun, Jun 2, 2:09 PM

This revision is now accepted and ready to land.Sun, Jun 2, 2:09 PM

rebase and land

Harbormaster completed remote builds in B29390: Diff 40889.Mon, Jun 3, 11:42 AM

Closed by commit rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock` (authored by atul). · Explain WhyMon, Jun 3, 12:08 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

4 lines

Diff 40889

View Options

lib/shared/thread-utils.js

[lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`ClosedPublicActions