Skip to content

LG-11270 Remove obsolete too much mail message on enter password controller#9434

Merged
soniaconnolly merged 1 commit intomainfrom
abrad-lg-11270-remove-obsolete-too-much-mail
Oct 23, 2023
Merged

LG-11270 Remove obsolete too much mail message on enter password controller#9434
soniaconnolly merged 1 commit intomainfrom
abrad-lg-11270-remove-obsolete-too-much-mail

Conversation

@theabrad
Copy link
Contributor

changelog: User-Facing Improvements, IdV, remove obsolete too much mail message

🎫 Ticket

LG-11270

🛠 Summary of changes

We removed the too much mail flash error message on the enter password controller.

📜 Testing Plan

If testing locally add these config variables

minimum_wait_before_another_usps_letter_in_hours: 0
max_mail_events: 1
  • Start IdV
  • Choose verify by mail
  • Request the maximum number of letters allowed (1, with the settings above)
  • Cancel and restart IdV
  • Verify by phone
  • Check to see that there is no error banner on the re-enter password step

changelog: User-Facing Improvements, IdV, remove obsolete too much mail
message

Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
@theabrad theabrad requested a review from a team October 23, 2023 19:31
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

LGTM. Confirmed locally that with the suggested config settings, the obsolete error message shows up on main, and doesn't show up on this branch.

@soniaconnolly soniaconnolly merged commit 9e48c52 into main Oct 23, 2023
@soniaconnolly soniaconnolly deleted the abrad-lg-11270-remove-obsolete-too-much-mail branch October 23, 2023 22:59
@mdiarra3 mdiarra3 mentioned this pull request Oct 26, 2023
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.

2 participants