[web] Return dispatchCreateSubchannel directly in Stepper.Item nextProps
ClosedPublic
Actions

Authored by ashoat on Dec 4 2023, 1:10 PM.

Details

Reviewers

tomek

Commits

rCOMM37f8ead8e105: [web] Return dispatchCreateSubchannel directly in Stepper.Item nextProps

Summary

While working on the unused-promise Flow lint, I noticed that we can simplify this code if we adjust the API in the Stepper component to allow mixed instead of void returns.

Depends on D10162

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Dec 4 2023, 1:10 PM

ashoat added a child revision: D10164: [web] Don't handle promise for onDeviceTokenRegistered call in PushNotifsHandler.

Harbormaster completed remote builds in B24742: Diff 34205.Dec 4 2023, 2:28 PM

ashoat requested review of this revision.Dec 4 2023, 2:28 PM

tomek accepted this revision.Dec 5 2023, 5:07 AM

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

Closed by commit rCOMM37f8ead8e105: [web] Return dispatchCreateSubchannel directly in Stepper.Item nextProps (authored by ashoat). · Explain WhyDec 5 2023, 12:32 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM37f8ead8e105: [web] Return dispatchCreateSubchannel directly in Stepper.Item nextProps.

Revision Contents
Changeset List

Path

Size

web/

components/

stepper.react.js

2 lines

modals/

threads/

create/

compose-subchannel-modal.react.js

4 lines

Diff 34298

View Options

web/components/stepper.react.js