-
Notifications
You must be signed in to change notification settings - Fork 62
FORMS-18328 default date localization changes #1722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add test case for the change
This is a CQ localization text change, but the localized text is still not available in the repository. Adding a manual library for the test case would defeat the purpose of testing, as it would ensure the test never fails. |
...oot/apps/core/fd/components/form/datepicker/v1/datepicker/clientlibs/editor/js/editDialog.js
Outdated
Show resolved
Hide resolved
@im-shiv Have we raised a PR in i18n repo as well - with the comment - to help localization team with the changes/context. |
...oot/apps/core/fd/components/form/datepicker/v1/datepicker/clientlibs/editor/js/editDialog.js
Outdated
Show resolved
Hide resolved
...oot/apps/core/fd/components/form/datepicker/v1/datepicker/clientlibs/editor/js/editDialog.js
Show resolved
Hide resolved
6a1b076
to
4ad6d5d
Compare
Here is the ticket for same: FORMS-22038 |
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
* FORMS-18328 default date localization changes * FORMS-18328 updating comment message --------- Co-authored-by: Shivam Agarwal <shivama@adobe.com>
No description provided.