Skip to content

Show "Come Back Later" as final screen with GPO + IPP#6596

Merged
aduth merged 5 commits intomainfrom
aduth-ipp-gpo-precedence
Jul 18, 2022
Merged

Show "Come Back Later" as final screen with GPO + IPP#6596
aduth merged 5 commits intomainfrom
aduth-ipp-gpo-precedence

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Jul 18, 2022

Why: A user who completes the in-person proofing flow while opting to verify their address by receiving a letter (GPO) should be shown the "Come Back Later" screen at the conclusion of the flow, and not be shown instructions directing them to proof in person.

This is a correction to the behavior implemented in #6576. Previously, a user would always be redirected to "Ready to verify", regardless of how they had selected to verify their address.

aduth added 5 commits July 18, 2022 08:46
So it can be easily reused
**Why**: A user who completes the in-person proofing flow while opting to verify their address by receiving a letter (GPO) should be shown the "Come Back Later" screen at the conclusion of the flow, and not be shown instructions directing them to proof in person.

changelog: Upcoming Features, In-Person Proofing, Show "Come Back Later" as final screen when opting to verify address by mail
@aduth aduth requested a review from a team July 18, 2022 13:21
Copy link
Contributor

@tomas-nava tomas-nava left a comment

Choose a reason for hiding this comment

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

approve!

Comment on lines +73 to +80
def complete_all_in_person_proofing_steps(user = user_with_2fa)
complete_location_step(user)
complete_prepare_step(user)
complete_state_id_step(user)
complete_address_step(user)
complete_ssn_step(user)
complete_verify_step(user)
end
Copy link
Contributor

Choose a reason for hiding this comment

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

thank you for making these!

@aduth aduth merged commit 0efbbe4 into main Jul 18, 2022
@aduth aduth deleted the aduth-ipp-gpo-precedence branch July 18, 2022 20:01
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