-
Notifications
You must be signed in to change notification settings - Fork 8
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: dates sent as ISO not DD-MM-YYYY #122
Conversation
@mamico This needs collective.volto.formsupport >= 3.1.4 (specifically collective/collective.volto.formsupport#34) to work right? What happens if the backend is less than this version? |
In this case, I believe dates would be saved/sent in ISO format instead of DD/MM/YYYY format. |
@mamico This is a breaking change then, right? Otherwise, LGTM! |
Not sure if we can call it breaking, but we can document the backend requirement in the README, as we've done for the others. |
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'll push these commits. This will make it easier to read and maintain.
Make a pretty list
Here's how the changes I made will look. Note
|
dates should be sent in ISO format (the widget's default should be fine), not DD-MM-YYYY