From 6a510815ee354b8a3a6e4519a8bd3256cf35f8f7 Mon Sep 17 00:00:00 2001 From: James McKinney <26463+jpmckinney@users.noreply.github.com> Date: Tue, 5 Dec 2023 22:41:55 -0500 Subject: [PATCH] chore: Remove unnecessary .name for StrEnum enums --- app/dependencies.py | 2 +- app/routers/applications.py | 2 +- app/routers/downloads.py | 4 +--- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/app/dependencies.py b/app/dependencies.py index ffc0dc19..4e577bad 100644 --- a/app/dependencies.py +++ b/app/dependencies.py @@ -97,7 +97,7 @@ def raise_if_unauthorized( if application.status not in statuses: raise HTTPException( status_code=status.HTTP_409_CONFLICT, - detail=f"Application status should not be {application.status.name}", + detail=f"Application status should not be {application.status}", ) diff --git a/app/routers/applications.py b/app/routers/applications.py index d94a4333..816b677a 100644 --- a/app/routers/applications.py +++ b/app/routers/applications.py @@ -177,7 +177,7 @@ async def approve_application( not_validated_documents = [] for document in application.borrower_documents: if not document.verified: - not_validated_documents.append(document.type.name) + not_validated_documents.append(document.type) if not_validated_documents: logger.error("Following documents were not validated: %s", not_validated_documents) raise HTTPException( diff --git a/app/routers/downloads.py b/app/routers/downloads.py index 70a05053..b62e58ca 100644 --- a/app/routers/downloads.py +++ b/app/routers/downloads.py @@ -167,9 +167,7 @@ async def export_applications( get_translated_string("Legal Name", lang): application.borrower.legal_name, get_translated_string("Email Address", lang): application.primary_email, get_translated_string("Submission Date", lang): application.borrower_submitted_at, - get_translated_string("Stage", lang): get_translated_string( - application.status.name.capitalize(), lang - ), + get_translated_string("Stage", lang): get_translated_string(application.status.capitalize(), lang), } )