Skip to content

LG-11260: Not ready section fix#9578

Merged
dawei-nava merged 1 commit intomainfrom
dwang/LG-11260-not-ready-cancel
Nov 15, 2023
Merged

LG-11260: Not ready section fix#9578
dawei-nava merged 1 commit intomainfrom
dwang/LG-11260-not-ready-cancel

Conversation

@dawei-nava
Copy link
Contributor

@dawei-nava dawei-nava commented Nov 12, 2023

🎫 Ticket

LG-11260: Fix account url, language preference.

🛠 Summary of changes

Not ready section cancel link should keep language preference.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Step 1: Enter doc auth flow, using no english language without SP
  • Step 2: On the doc capture pages
  • Step 3: Click on the related cancel in the not ready section
  • Step 4: It should go back to account home.

@dawei-nava dawei-nava force-pushed the dwang/LG-11260-not-ready-cancel branch from ad4e244 to f193e8b Compare November 12, 2023 20:30
…reference).

changelog: User-Facing Improvements, Doc Auth, Return to account home.
@dawei-nava dawei-nava force-pushed the dwang/LG-11260-not-ready-cancel branch from f193e8b to 24597db Compare November 12, 2023 20:32
@dawei-nava dawei-nava marked this pull request as ready for review November 13, 2023 01:37
Copy link
Contributor

@amirbey amirbey left a comment

Choose a reason for hiding this comment

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

@dawei-nava - this works as expected in accord with the testing plan ... can we add tests to check the behavior for SP vs no SP?

@dawei-nava
Copy link
Contributor Author

@dawei-nava - this works as expected in accord with the testing plan ... can we add tests to check the behavior for SP vs no SP?

@amirbey , those tests are already implemented in document-capture-not-ready-spec.tsx

@amirbey amirbey self-requested a review November 15, 2023 15:48
Copy link
Contributor

@amirbey amirbey left a comment

Choose a reason for hiding this comment

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

LGTM

@dawei-nava dawei-nava merged commit ee8c8c6 into main Nov 15, 2023
@dawei-nava dawei-nava deleted the dwang/LG-11260-not-ready-cancel branch November 15, 2023 15:49
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