[web] Simplify `return` value of `changeThreadSettingsAction`
ClosedPublic
Actions

Authored by • abosh on Aug 23 2022, 9:34 PM.

Details

Reviewers

tomek
atul

Commits

rCOMM669f84ba59ec: [web] Simplify `return` value of `changeThreadSettingsAction`

Summary

Noticed this when reading through @tomek's feedback on D4416 when requesting review on D4931. Looks like I forgot to add this in after he accepted!

Test Plan

Flow, @tomek's feedback

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• abosh created this revision.Aug 23 2022, 9:34 PM

Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2022, 9:34 PM

• abosh edited the summary of this revision. (Show Details)Aug 23 2022, 9:34 PM

Harbormaster completed remote builds in B11557: Diff 15892.Aug 23 2022, 9:43 PM

• abosh requested review of this revision.Aug 23 2022, 9:43 PM

atul accepted this revision.Aug 26 2022, 5:16 AM

This revision is now accepted and ready to land.Aug 26 2022, 5:16 AM

Closed by commit rCOMM669f84ba59ec: [web] Simplify `return` value of `changeThreadSettingsAction`. · Explain WhyAug 26 2022, 5:28 PM

This revision was automatically updated to reflect the committed changes.

• abosh added a commit: rCOMM669f84ba59ec: [web] Simplify `return` value of `changeThreadSettingsAction`.

Revision Contents
Changeset List

Path

Size

web/

modals/

threads/

settings/

thread-settings-general-tab.react.js

3 lines

Diff 15892

View Options

web/modals/threads/settings/thread-settings-general-tab.react.js

[web] Simplify `return` value of `changeThreadSettingsAction`ClosedPublicActions