Skip to content

LG-15179 Timed out hybrid user should be able to retry docv#11792

Merged
theabrad merged 3 commits intomainfrom
abrad-lg-15179-timeout-retry-docv
Jan 29, 2025
Merged

LG-15179 Timed out hybrid user should be able to retry docv#11792
theabrad merged 3 commits intomainfrom
abrad-lg-15179-timeout-retry-docv

Conversation

@theabrad
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket:
LG-15179

🛠 Summary of changes

This fix makes sure a user is allowed to complete DocV after they timeout and the timeout is over.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

changelog: Internal, IdV Doc Auth, Allow timed out user to retry docv
Copy link
Contributor

@AShukla-GSA AShukla-GSA left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

Can we include some test coverage for this behavior?

@theabrad theabrad merged commit 6fa0057 into main Jan 29, 2025
2 checks passed
@theabrad theabrad deleted the abrad-lg-15179-timeout-retry-docv branch January 29, 2025 16:26
n1zyy pushed a commit that referenced this pull request Jan 31, 2025
* set socure_docv_wait_polling timestamp to nil in show on hybrid mobile

* add changelog

changelog: Internal, IdV Doc Auth, Allow timed out user to retry docv

* have spec ensure socure_docv_wait_polling_started_at is nil
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