open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
37 stars 26 forks source link

:bug: [#4772] Add migration to set select dataType to string #4818

Closed stevenbal closed 1 week ago

stevenbal commented 1 week ago

because Select components did not have a dataType set, formio tries to cast it to other types if possible, which causes issues when submitting the data to the backend. For that reason we set the value to string to avoid this unwanted normalization

TODO:

Closes #4772

Changes

[Describe the changes here]

Checklist

Check off the items that are completed or not relevant.

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.55%. Comparing base (b5daa22) to head (0fe69b3). Report is 5 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #4818 +/- ## ======================================= Coverage 96.55% 96.55% ======================================= Files 748 748 Lines 25417 25423 +6 Branches 3361 3362 +1 ======================================= + Hits 24542 24548 +6 Misses 610 610 Partials 265 265 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sergei-maertens commented 1 week ago

Not backporting this PR, instead we'll backport the fix via the SDK