Merge pull request #320 from dod-ccpo/modal-redirects-#160668407

fix financial verification modal bug and refactor modal redirects

atst/routes/requests/financial_verification.py

@@ -72,7 +72,7 @@ def update_financial_verification(request_id):
             else:
                 return redirect(
                     url_for(
-                        "requests.requests_index", pendingFinancialVerification=True
+                        "requests.requests_index", modal="pendingFinancialVerification"
                     )
                 )
 

atst/routes/requests/requests_form.py

@@ -119,10 +119,11 @@ def requests_submit(request_id=None):
     Requests.submit(request)
 
     if request.status == RequestStatus.PENDING_FINANCIAL_VERIFICATION:
-        return redirect("/requests?modal=pendingFinancialVerification")
-
+        modal = "pendingFinancialVerification"
     else:
-        return redirect("/requests?modal=pendingCCPOApproval")
+        modal = "pendingCCPOApproval"
+
+    return redirect(url_for("requests.requests_index", modal=modal))
 
 
 @requests_bp.route("/requests/details/<string:request_id>", methods=["GET"])