Skip to content
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

Merged
merged 4 commits into from
Jan 24, 2024
Merged

Conversation

SilviaAmAm
Copy link
Contributor

@SilviaAmAm SilviaAmAm commented Jan 18, 2024

Fixes #3755

Front end PR: open-formulieren/open-forms-sdk#636

Related issue for the date component: #3443

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (970fb94) 96.40% compared to head (3072601) 96.40%.

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.
📢 Have feedback on the report? Share it here.

@SilviaAmAm SilviaAmAm changed the title [#3577] Fix datetime validation [#3755] Fix datetime validation Jan 18, 2024
@SilviaAmAm SilviaAmAm force-pushed the fix/3577-datetime-validation branch from 5be9b86 to a6c00a4 Compare January 18, 2024 15:15
@vaszig vaszig self-assigned this Jan 18, 2024
Copy link
Contributor

@vaszig vaszig left a 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.

@SilviaAmAm
Copy link
Contributor Author

The validation will be handled in the SDK right?I mean the shown error.

Yes!

@SilviaAmAm
Copy link
Contributor Author

SilviaAmAm commented Jan 22, 2024

To figure out: do we need to do something for the new form builder?

Answer: yes

@SilviaAmAm SilviaAmAm force-pushed the fix/3577-datetime-validation branch from a6c00a4 to 14cef02 Compare January 23, 2024 11:28
@SilviaAmAm SilviaAmAm force-pushed the fix/3577-datetime-validation branch from 14cef02 to 3072601 Compare January 23, 2024 11:30
Copy link
Member

@sergei-maertens sergei-maertens left a 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.

@sergei-maertens sergei-maertens merged commit 507b3c6 into master Jan 24, 2024
27 checks passed
@sergei-maertens sergei-maertens deleted the fix/3577-datetime-validation branch January 24, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Form builder - Datetime field empties itself when a invalid date is entered
3 participants