[lib] Get rid of switchMemberAdminRoleInThread
ClosedPublic
Actions

Authored by ashoat on Dec 4 2023, 9:08 PM.

Details

Reviewers

rohan

Commits

rCOMM30126e729887: [lib] Get rid of switchMemberAdminRoleInThread

Summary

Working on enabling the unused-promise Flow lint (ENG-5907), and this got flagged because of the dispatchActionPromise usage.

I looked but I couldn't see any usages of this function in the codebase. I think the last usage was removed in D8155.

Depends on D10158

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Dec 4 2023, 9:08 PM

Herald added a subscriber: tomek. · View Herald TranscriptDec 4 2023, 9:08 PM

ashoat added a child revision: D10160: [lib] Don't return a promise from ignorePromiseRejections.Dec 4 2023, 9:09 PM

Harbormaster completed remote builds in B24738: Diff 34201.Dec 4 2023, 9:39 PM

ashoat requested review of this revision.Dec 4 2023, 9:39 PM

rohan accepted this revision.Dec 5 2023, 3:26 AM

This revision is now accepted and ready to land.Dec 5 2023, 3:26 AM

Closed by commit rCOMM30126e729887: [lib] Get rid of switchMemberAdminRoleInThread. · Explain WhyDec 5 2023, 8:32 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM30126e729887: [lib] Get rid of switchMemberAdminRoleInThread.

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

34 lines

Diff 34201

View Options

lib/shared/thread-utils.js

[lib] Get rid of switchMemberAdminRoleInThreadClosedPublicActions