Skip to content

LG-9016 Always show Address Line 2 field in Verify Info screen#7916

Merged
soniaconnolly merged 2 commits intomainfrom
sonia-lg-9016-verify-info-address-line2
Mar 2, 2023
Merged

LG-9016 Always show Address Line 2 field in Verify Info screen#7916
soniaconnolly merged 2 commits intomainfrom
sonia-lg-9016-verify-info-address-line2

Conversation

@soniaconnolly
Copy link
Contributor

🎫 Ticket

LG-9016

🛠 Summary of changes

On Verify Info screen, show Address Line2 even if it is currently blank. This lets the user split their address into two lines if needed, e.g. with Puerto Rico urbanizations.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Create an account
  • Navigate to /verify
  • Go through proofing, and confirm Verify Info page behaves correctly with and without an Address line 2
  • Edit the address to add and remove Line 2

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.

Before:

Without Address LIne 2

After:

With blank Address Line 2

This lets the user split their address into two lines if needed,
e.g. with Puerto Rico urbanizations.

changelog: User-Facing Improvements, Identity verification, On Verify Info page always show Address Line 2 field
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

@soniaconnolly soniaconnolly requested a review from a team March 2, 2023 20:08
TIL about .presence
Copy link
Contributor

@theabrad theabrad left a comment

Choose a reason for hiding this comment

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

LGTM

@soniaconnolly soniaconnolly merged commit 6c64aa9 into main Mar 2, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-9016-verify-info-address-line2 branch March 2, 2023 21:42
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