Skip to content

LG-11753 Update Enter Code (formerly, Welcome Back) Screen#10344

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-redo-enter-gpo-code-screen
May 6, 2024
Merged

LG-11753 Update Enter Code (formerly, Welcome Back) Screen#10344
jmhooper merged 2 commits intomainfrom
jmhooper-redo-enter-gpo-code-screen

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Apr 1, 2024

This commit makes changes to the Enter Code screen in the welcome back flow. It updates the address and “Clear my information and start over” link to be in an accordion, makes content updates, and rearranges some of the elements.

@jmhooper
Copy link
Contributor Author

jmhooper commented Apr 1, 2024

Here are screenshots of the updated enter-code screen:

English:
image

English with alerts:
image

Spanish:
image

Spanish with alerts:
image

French:
image

French with alerts:
image

@jmhooper
Copy link
Contributor Author

jmhooper commented Apr 1, 2024

It looks like moving the "Clear my information and start over" link into the accordion has caused some issues with feature specs. I am going to go through and tackle those.

@jmhooper jmhooper force-pushed the jmhooper-redo-enter-gpo-code-screen branch 2 times, most recently from 094d9ef to e856b34 Compare April 1, 2024 20:33
@jmhooper
Copy link
Contributor Author

jmhooper commented Apr 1, 2024

The Accordion Body in the screenshots is actually not correct. Where is says "Where was my letter sent?" it should say "We sent a letter with your verification code to:". I have fixed that issue.

@jmhooper jmhooper requested a review from a team April 1, 2024 21:01
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.

Ran it locally. Screenshots matched. LGTM

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.

Looking at the AC: The "Confirm account" button should be changed to "Submit"

@jmhooper
Copy link
Contributor Author

jmhooper commented Apr 3, 2024

@theabrad: Updated in ab623ce. I'll need to capture new screenshots.

This commit makes changes to the Enter Code screen in the welcome back flow. It updates the address and “Clear my information and start over” link to be in an accordion, makes content updates, and rearranges some of the elements.

[skip changelog]
@jmhooper jmhooper force-pushed the jmhooper-redo-enter-gpo-code-screen branch from ab623ce to 1856943 Compare May 6, 2024 14:18
@jmhooper jmhooper merged commit a890186 into main May 6, 2024
@jmhooper jmhooper deleted the jmhooper-redo-enter-gpo-code-screen branch May 6, 2024 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants