-
Notifications
You must be signed in to change notification settings - Fork 26
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
[#3755] Fix datetime validation #3787
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3787 +/- ##
=======================================
Coverage 96.40% 96.40%
=======================================
Files 706 706
Lines 22182 22185 +3
Branches 2534 2534
=======================================
+ Hits 21385 21388 +3
Misses 549 549
Partials 248 248 ☔ View full report in Codecov by Sentry. |
5be9b86
to
a6c00a4
Compare
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 validation will be handled in the SDK right?I mean the shown error.
Yes! |
To figure out: do we need to do something for the new form builder? Answer: yes |
a6c00a4
to
14cef02
Compare
14cef02
to
3072601
Compare
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 need to properly incorporate this in the squashed migrations for 2.5.0
Contrary to my first remarks, I believe it will be easier to manage everything by not postponing this for 2.5.1 due to the involved migrations.
Fixes #3755
Front end PR: open-formulieren/open-forms-sdk#636
Related issue for the date component: #3443