Page MenuHomePhabricator

Add a warning about MFA requirements to edit forms
ClosedPublic

Authored by epriestley on Jan 28 2019, 6:29 PM.
Tags
None
Referenced Files
F13098796: D20045.diff
Fri, Apr 26, 10:13 AM
Unknown Object (File)
Mon, Apr 8, 2:47 PM
Unknown Object (File)
Mon, Apr 8, 12:02 PM
Unknown Object (File)
Sun, Apr 7, 7:52 PM
Unknown Object (File)
Mar 27 2024, 5:20 PM
Unknown Object (File)
Mar 23 2024, 12:32 PM
Unknown Object (File)
Mar 21 2024, 9:10 AM
Unknown Object (File)
Feb 15 2024, 9:49 PM
Subscribers
None

Details

Summary

Depends on D20044. Ref T13242. Similar to D20044, add reminder text to edit forms.

It would be nice to "workflow" these so the MFA flow happens inline, but Maniphest's inline edit behavior currently conflicts with this. Set it aside for now since the next workboards iteration (triggers) is probably a good opportunity to revisit it.

Test Plan

Screen Shot 2019-01-28 at 10.27.40 AM.png (329×699 px, 26 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This change looks good, but what exactly does it accomplish? I feel like this would be useful if a user was trying to take an action that required MFA, but their account had no MFA factors configured, but is such a thing even possible? Otherwise this feels like a message that we train users to ignore whenever they see it.

Oh, there's more context in D20044, hang on.

This revision is now accepted and ready to land.Jan 29 2019, 2:38 AM

Yeah, D20044 is the better version of this.

This is just a hint like "go dig your phone out of your bag before writing your entire memoir into this box". The warnings in D20044 are more substantive/useful.

This revision was automatically updated to reflect the committed changes.