Differential D19896

In Legalpad, prompt for MFA at the end of the workflow instead of the beginning
ClosedPublic
Actions

Authored by epriestley on Dec 17 2018, 7:23 PM.

Tags

None

Referenced Files

	F15552792: D19896.id47504.diff
	Sun, Apr 27, 10:39 PM

	F15551793: D19896.diff
	Sun, Apr 27, 6:36 PM

	F15535827: D19896.id47504.diff
	Thu, Apr 24, 7:00 AM

	F15531942: D19896.diff
	Wed, Apr 23, 1:56 PM

	F15526319: D19896.diff
	Mon, Apr 21, 8:35 PM

	F15520280: D19896.id47505.diff
	Sun, Apr 20, 5:16 AM

	F15512540: D19896.id47505.diff
	Thu, Apr 17, 11:26 AM

	F15512535: D19896.id47504.diff
	Thu, Apr 17, 11:24 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13222: 2018 Week 48-51 Bonus Content

Commits

rP961fd7e8491e: In Legalpad, prompt for MFA at the end of the workflow instead of the beginning

Summary

Depends on D19895. Ref T13222. This is a simple behavioral improvement for the current MFA implementation in Legalpad: don't MFA the user and then realize that they forgot to actually check the box.

Test Plan

Submitted form without the box checked, got an error saying "check the box" instead of MFA.
Submitted the form with the box checked, got an MFA prompt.
Passed the MFA gate, got a signed form.
Tried to sign another form, hit MFA timed lockout.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Actual change which does what I claim.

Harbormaster completed remote builds in B21358: Diff 47504.Dec 17 2018, 7:26 PM

Harbormaster completed remote builds in B21359: Diff 47505.

epriestley requested review of this revision.Dec 17 2018, 7:26 PM

epriestley added a child revision: D19897: Allow any transaction group to be signed with a one-shot "Sign With MFA" action.Dec 17 2018, 8:37 PM

amckinley accepted this revision.Dec 18 2018, 8:50 PM

This revision is now accepted and ready to land.Dec 18 2018, 8:50 PM

Closed by commit rP961fd7e8491e: In Legalpad, prompt for MFA at the end of the workflow instead of the beginning (authored by epriestley). · Explain WhyDec 20 2018, 10:51 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

legalpad/

controller/

LegalpadDocumentSignController.php

29 lines

Diff 47557

src/applications/legalpad/controller/LegalpadDocumentSignController.php

Loading...