Page MenuHomePhabricator
Differential D3840

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

Authored by atul on Apr 25 2022, 11:38 AM.
Tags
None
Referenced Files
F3376736: D3840.id12086.diff
Wed, Nov 27, 1:55 AM
Unknown Object (File)
Sat, Nov 23, 9:46 AM
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
View All Files
Subscribers
• abosh
• adrian
ashoat
• karol

Details

Reviewers
• jacek
tomek
Commits
rCOMMe9ae786729d6: [web] Push `accountPassword` state and related functionality down to…
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Apr 25 2022, 11:38 AM
Herald added subscribers: • abosh, • karol, • adrian, ashoat. · View Herald TranscriptApr 25 2022, 11:38 AM
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!
Harbormaster completed remote builds in B8527: Diff 11886.Apr 25 2022, 11:43 AM
atul requested review of this revision.Apr 25 2022, 11:43 AM
atul added a child revision: D3842: [web] Push `changeThreadSettingsAction` down to `ThreadSettingsGeneralTab`.Apr 25 2022, 12:39 PM
tomek accepted this revision.Apr 26 2022, 8:54 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
atul updated this revision to Diff 11954.Apr 26 2022, 10:46 AM

rebase

Harbormaster completed remote builds in B8575: Diff 11954.Apr 26 2022, 10:52 AM
atul updated this revision to Diff 12085.Apr 28 2022, 2:33 PM

rebase before landing

Harbormaster completed remote builds in B8678: Diff 12085.Apr 28 2022, 2:37 PM
Closed by commit rCOMMe9ae786729d6: [web] Push `accountPassword` state and related functionality down to… (authored by atul). · Explain WhyApr 28 2022, 2:43 PM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMe9ae786729d6: [web] Push `accountPassword` state and related functionality down to….

Revision Contents
Changeset List

Diff 12086

View Options

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

Loading...
View Options

web/modals/threads/thread-settings-modal.react.js

Loading...