Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lg-14704 refactor doc capture routing #11897

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

AShukla-GSA
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket:
LG-14704

🛠 Summary of changes

Added method to correctly choose correct vendor controller and calls the function to directly redirect to the correct vendor controller instead of using the standard document capture controller to redirect to the correct vendor.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Step 1 - Ensure added spec changes adequately tests new logic

@amirbey
Copy link
Contributor

amirbey commented Feb 19, 2025

the intention of this ticket is for users to be routed directly to the correct document capture controller. currently all standard flow users are routed (from here and here ... also check for others) directly to the mock/LN and then forwarded to Socure document capture if their doc_auth_vendor is Socure.

i'm wondering if there is an opportunity to reuse redirect_to_correct_vendor instead of always sending a user to the mock/LN document capture controller 🤔

@AShukla-GSA AShukla-GSA requested a review from amirbey February 20, 2025 16:54
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