Page MenuHomePhabricator

[web] Push `accountPassword` state and related functionality down to `ThreadSettingsDeleteTab`
ClosedPublic

Authored by atul on Apr 25 2022, 11:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 13, 1:19 AM
Unknown Object (File)
Sat, Nov 9, 10:48 PM
Unknown Object (File)
Thu, Nov 7, 10:58 PM
Unknown Object (File)
Thu, Nov 7, 10:17 PM
Unknown Object (File)
Thu, Nov 7, 7:53 PM
Unknown Object (File)
Tue, Nov 5, 4:45 AM
Unknown Object (File)
Oct 16 2024, 8:41 AM
Unknown Object (File)
Oct 11 2024, 7:09 PM

Details

Summary

We don't need to be holding onto this state in the parent component and passing it down to ThreadSettingsDeleteTab as a prop.


Depends on D3838

Test Plan

Careful reading, just drag and drop

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D3840 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 25 2022, 11:38 AM
Harbormaster failed remote builds in B8527: Diff 11886!
atul requested review of this revision.Apr 25 2022, 11:43 AM
tomek added inline comments.
web/modals/threads/thread-settings-modal.react.js
151 ↗(On Diff #11886)

Please make sure that after failing to submit the changes, and opening delete tab, the input is cleared.

This revision is now accepted and ready to land.Apr 26 2022, 8:54 AM