-
-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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
fix(scheduling): update error message #20208
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…g-update-error-message
…g-update-error-message
…g-update-error-message
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.
I guess we're moving the validations because the backend wasn't using them?
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.
The backend use some of them but with different fields/formats, also in the front we want to throw translated errors and we can't do that on back so one for both don't work anymore
What does it do?
Updates the scheduling required field error message to use more generic required field message.
Why is it needed?
Asked by UX, https://www.notion.so/strapi/Releases-Scheduling-Alpha-Blitz-Session-14ae56786d5a41e2a0dbf98b7e4b55e2?pvs=4#8916a056719f4dbb862c479bf1eab087
How to test it?
When user tries to schedule a release if any of the required field is not filled, on submit user should get error message This value is required.