From 41bd0351320083b6829a951eb86bf3cd1d6c07f6 Mon Sep 17 00:00:00 2001 From: Eugene Dyudyunov Date: Wed, 28 Feb 2024 10:23:15 +0200 Subject: [PATCH] fix: limit email subject length Add limitation for subject length (128 chars) and the corresponding help text. --- .../bulk-email-tool/bulk-email-form/BulkEmailForm.jsx | 5 ++++- src/components/bulk-email-tool/bulk-email-form/messages.js | 5 +++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/components/bulk-email-tool/bulk-email-form/BulkEmailForm.jsx b/src/components/bulk-email-tool/bulk-email-form/BulkEmailForm.jsx index eb8dc33..aafd53c 100644 --- a/src/components/bulk-email-tool/bulk-email-form/BulkEmailForm.jsx +++ b/src/components/bulk-email-tool/bulk-email-form/BulkEmailForm.jsx @@ -281,7 +281,10 @@ function BulkEmailForm(props) { /> {intl.formatMessage(messages.bulkEmailSubjectLabel)} - + + + {intl.formatMessage(messages.bulkEmailFormSubjectTip)} + {!emailFormValidation.subject && ( {intl.formatMessage(messages.bulkEmailFormSubjectError)} diff --git a/src/components/bulk-email-tool/bulk-email-form/messages.js b/src/components/bulk-email-tool/bulk-email-form/messages.js index c304c07..0d0194a 100644 --- a/src/components/bulk-email-tool/bulk-email-form/messages.js +++ b/src/components/bulk-email-tool/bulk-email-form/messages.js @@ -41,6 +41,11 @@ const messages = defineMessages({ defaultMessage: 'Subject', description: 'Email subject line input label. Meant to have colon or equivilant punctuation.', }, + bulkEmailFormSubjectTip: { + id: 'bulk.email.form.subject.tip', + defaultMessage: '(Maximum 128 characters)', + description: 'Default Subject tip', + }, bulkEmailFormSubjectError: { id: 'bulk.email.form.subject.error', defaultMessage: 'A subject is required',