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

	F18985923: D19896.id47504.diff
	Mon, Nov 17, 1:47 PM

	F18971507: D19896.id47504.diff
	Sat, Nov 15, 12:16 PM

	F18876972: D19896.diff
	Thu, Nov 6, 5:08 AM

	F18863728: D19896.diff
	Mon, Nov 3, 12:51 AM

	F18764173: D19896.id.diff
	Oct 7 2025, 4:40 AM

	F18746881: D19896.id.diff
	Oct 3 2025, 1:49 PM

	F18742757: D19896.diff
	Oct 2 2025, 9:16 PM

	F18673234: D19896.diff
	Sep 25 2025, 9:50 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...