i18n: Required fields text to "{Field} required" format #33077
Conversation
…on text Added a new text for required fields that follows design specifications. Additionally, I updated the Direct_message_creation_error text to better align with the new guidelines.
|
|
Looks like this PR is ready to merge! 🎉 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #33077 +/- ##
===========================================
+ Coverage 59.68% 59.71% +0.02%
===========================================
Files 2551 2551
Lines 62819 62823 +4
Branches 14115 14107 -8
===========================================
+ Hits 37496 37515 +19
+ Misses 22908 22901 -7
+ Partials 2415 2407 -8
Flags with carried forward coverage won't be shown. Click here to find out more. |
Changed all places that used alternative forms of the "The_field_is_required" i18n string to it and refactored some forms to properly use the useForm errors object.
…' and properly implement useForm In addition to updating the required field texts in other forms, this commit includes several refactors: the mode of useForm was changed to 'all', unique IDs were introduced where needed, inputs were refactored to improve the use of the RHF controller, and the disabled attribute was removed from submit buttons
…itDepartment wrong translation Adapted the omnichannel tests to the new required field text and fixed a misuse of the 'The_field_is_required' translation string in which untranslated lowercase strings of the given field name were being passed to the i18n t function.
|
dionisio-bot
bot
added
stat: ready to merge
kodiakhq
bot
removed
the
stat: ready to merge
|
This PR currently has a merge conflict. Please resolve this and then re-add the |
rique223
changed the title
rique223
changed the title
Proposed changes (including videos or screenshots)
Currently, mandatory fields display the following error message in production: 'The field %s is required.' For example: 'The field email is required.' Product has decided to change the format of this message to '%s required' For example: 'Email required' Since this is the default 'Field required' message, updating it across all areas of the system is necessary. This PR aims to address that change.
Demo gif:
Issue(s)
Steps to test or reproduce
Further comments
Jira task: CONN-314