diff --git a/atst/domain/requests.py b/atst/domain/requests.py index 06f1e24f..93cbac90 100644 --- a/atst/domain/requests.py +++ b/atst/domain/requests.py @@ -268,8 +268,7 @@ WHERE requests_with_status.status = :status return request @classmethod - def submit_financial_verification(cls, request_id): - request = Requests._get_with_lock(request_id) + def submit_financial_verification(cls, request): Requests.set_status(request, RequestStatus.PENDING_CCPO_APPROVAL) db.session.add(request) diff --git a/atst/routes/requests/financial_verification.py b/atst/routes/requests/financial_verification.py index 50f04102..ab25a235 100644 --- a/atst/routes/requests/financial_verification.py +++ b/atst/routes/requests/financial_verification.py @@ -39,15 +39,17 @@ def update_financial_verification(request_id): existing_request.body.get("financial_verification") ) updated_request = Requests.update_financial_verification(request_id, form.data) - if valid and updated_request.is_financially_verified: - new_workspace = Requests.approve_and_create_workspace(updated_request) - return redirect( - url_for( - "workspaces.new_project", - workspace_id=new_workspace.id, - newWorkspace=True, + if valid: + submitted_request = Requests.submit_financial_verification(updated_request) + if updated_request.is_financially_verified: + new_workspace = Requests.approve_and_create_workspace(submitted_request) + return redirect( + url_for( + "workspaces.new_project", + workspace_id=new_workspace.id, + newWorkspace=True, + ) ) - ) else: form.reset()