From a7417b4f3941e1f4189e9fa73ce846f28d6cabd1 Mon Sep 17 00:00:00 2001 From: richard-dds Date: Wed, 14 Aug 2019 11:35:07 -0400 Subject: [PATCH] Add upload errors to translations file --- atst/forms/task_order.py | 4 ++-- templates/components/upload_input.html | 2 +- translations.yaml | 6 ++++++ 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/atst/forms/task_order.py b/atst/forms/task_order.py index db5c42e8..6422aee7 100644 --- a/atst/forms/task_order.py +++ b/atst/forms/task_order.py @@ -68,13 +68,13 @@ class AttachmentForm(BaseForm): filename = HiddenField( id="attachment_filename", validators=[ - Length(max=100, message="Filename may be no longer than 100 characters.") + Length(max=100, message=translate("forms.attachment.filename.length_error")) ], ) object_name = HiddenField( id="attachment_object_name", validators=[ - Length(max=40, message="Object name may be no longer than 40 characters.") + Length(max=40, message=translate("forms.attachment.object_name.length_error")) ], ) accept = ".pdf,application/pdf" diff --git a/templates/components/upload_input.html b/templates/components/upload_input.html index b18e4e9f..a56cb0bf 100644 --- a/templates/components/upload_input.html +++ b/templates/components/upload_input.html @@ -44,7 +44,7 @@ {% for error, error_messages in field.errors.items() %} {{error_messages[0]}} diff --git a/translations.yaml b/translations.yaml index fd97ae8f..2a50ab80 100644 --- a/translations.yaml +++ b/translations.yaml @@ -145,7 +145,13 @@ forms: portfolio: name_label: Portfolio name name_length_validation_message: Portfolio names can be between 4-100 characters + attachment: + object_name: + length_error: Object name may be no longer than 40 characters. + filename: + length_error: Filename may be no longer than 100 characters. task_order: + upload_error: There was an error uploading your file. Please try again. If you encounter repeated problems uploading this file, please contact CCPO. app_migration: both: 'Yes, migrating from both an on-premise data center and another cloud provider' cloud: 'Yes, migrating from another cloud provider'