Skip to content

LG-5079: Create separate throttle for address verification#5546

Merged
aduth merged 2 commits intomainfrom
aduth-lg-5079-phone-throttle
Oct 26, 2021
Merged

LG-5079: Create separate throttle for address verification#5546
aduth merged 2 commits intomainfrom
aduth-lg-5079-phone-throttle

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Oct 26, 2021

Why: As a user, I expect that if it takes me several attempts to complete the "Verify your information" verification during IAL2, my remaining attempts do not carry over subsequent into the phone/address verification, so that I have adequate opportunity to try to complete this step independent of my previous difficulty.

**Why**: As a user, I expect that if it takes me several attempts to complete the "Verify your information" verification during IAL2, my remaining attempts do not carry over subsequent into the phone/address verification, so that I have adequate opportunity to try to complete this step independent of my previous difficulty.
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

It wasn't working as expected, because the created throttle wasn't actually throttled. The controller will always assign a time for expires_at, even if that time is 'now'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants