diff --git a/app/controllers/concerns/idv/verify_info_concern.rb b/app/controllers/concerns/idv/verify_info_concern.rb index b1c8ce2f56b..fcfddf75b72 100644 --- a/app/controllers/concerns/idv/verify_info_concern.rb +++ b/app/controllers/concerns/idv/verify_info_concern.rb @@ -2,7 +2,7 @@ module Idv module VerifyInfoConcern extend ActiveSupport::Concern - def update + def shared_update return if idv_session.verify_info_step_document_capture_session_uuid analytics.idv_doc_auth_verify_submitted(**analytics_arguments) Funnel::DocAuth::RegisterStep.new(current_user.id, sp_session[:issuer]). @@ -48,15 +48,7 @@ def update double_address_verification: capture_secondary_id_enabled, ) - # Don't allow the user to go back to document capture after verifying - if flow_session['redo_document_capture'] - flow_session.delete('redo_document_capture') - flow_session[:flow_path] ||= 'standard' - end - - # Mark the FSM verify step completed. This is for the 50/50 state - flow_session['Idv::Steps::InPerson::VerifyStep'] = true - redirect_to after_update_url + return true end private diff --git a/app/controllers/idv/in_person/verify_info_controller.rb b/app/controllers/idv/in_person/verify_info_controller.rb index 9891320011c..98c07f42826 100644 --- a/app/controllers/idv/in_person/verify_info_controller.rb +++ b/app/controllers/idv/in_person/verify_info_controller.rb @@ -36,6 +36,17 @@ def show process_async_state(load_async_state) end + def update + success = shared_update + + if success + # Mark the FSM verify step completed. This is for the 50/50 state + flow_session['Idv::Steps::InPerson::VerifyStep'] = true + + redirect_to idv_in_person_verify_info_url + end + end + private # state_id_type is hard-coded here because it's required for proofing against @@ -50,10 +61,6 @@ def invalid_state? pii.blank? end - def after_update_url - idv_in_person_verify_info_url - end - def prev_url idv_in_person_step_url(step: :ssn) end diff --git a/app/controllers/idv/verify_info_controller.rb b/app/controllers/idv/verify_info_controller.rb index f88706d7c39..ac3afbbe809 100644 --- a/app/controllers/idv/verify_info_controller.rb +++ b/app/controllers/idv/verify_info_controller.rb @@ -34,15 +34,25 @@ def show process_async_state(load_async_state) end + def update + success = shared_update + + if success + # Don't allow the user to go back to document capture after verifying + if flow_session['redo_document_capture'] + flow_session.delete('redo_document_capture') + flow_session[:flow_path] ||= 'standard' + end + + redirect_to idv_verify_info_url + end + end + private # state ID type isn't manually set for Idv::VerifyInfoController def set_state_id_type; end - def after_update_url - idv_verify_info_url - end - def prev_url idv_ssn_url end diff --git a/app/views/idv/verify_info/new.html.erb b/app/views/idv/verify_info/new.html.erb deleted file mode 100644 index e69de29bb2d..00000000000