LG-9097 document capture show outside FSM#7926
Conversation
And default to false
and make specs pass
and make it pass changelog: Internal, refactoring Flow State Machine, add new DocumentCaptureController with show action for desktop flow
eric-gade
left a comment
eric-gade
left a comment
There was a problem hiding this comment.
Right now it doesn't seem like there is any checking about the previous steps being completed (when the feature flag is enabled).
Here is what I did -- all local dev:
- Created a new account
- went to localhost:3000/verify and loaded that page
- Manually changed the url to /verify/document_capture
I was able to see the document capture page in that case, which I didn't expect. I thought I would be thrown back to the beginning of the doc auth flow since I had not yet completed any of the previous steps.
Ignore this if that is part of the multi-PR rollout of the ticket.
| current_flow_step_counts['Idv::Steps::DocumentCaptureStep'] += 1 | ||
| end | ||
|
|
||
| def native_camera_ab_testing_variables |
There was a problem hiding this comment.
Not immediately relevant, but we should ask if we still even need this a/b test anymore
Let's check during refinement whether that should be part of this ticket. Thanks! |
|
@eric-gade Updating from refinement, we're adding an AC about checking if previous steps were completed to the next ticket. Good catch. |
3 participants
🎫 Ticket
LG-9097
🛠 Summary of changes
Add DocumentCaptureController that shows same view as DocumentCapture step for desktop flow, with supporting controller and feature tests. Functionality is behind feature flag
doc_auth_document_capture_controller_enabled, which defaults to false.📜 Testing Plan
Provide a checklist of steps to confirm the changes.
👀 Screenshots
After: