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

Fixes SelectMultipleField value coercion on validation. #823

Merged

Conversation

azmeuk
Copy link
Member

@azmeuk azmeuk commented Dec 29, 2023

Fixes the SelectMultipleField validation error when using coercion detailed in #822.

I also avoids to use set to temporarily store choice values, so there is no more hashing issue.

fixes #822

@azmeuk azmeuk added the bug Unexpected behavior label Dec 29, 2023
@azmeuk azmeuk merged commit c53d557 into pallets-eco:master Jan 6, 2024
6 of 8 checks passed
@azmeuk azmeuk deleted the issue-822-selectmultiplefield-coerce branch January 6, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected behavior
Development

Successfully merging this pull request may close these issues.

SelectMultipleField should support choices specified as coercible values.
1 participant