Skip to content

[release/8.0] Don't throw for non-generic collection types#32059

Merged
ajcvickers merged 1 commit intorelease/8.0from
231016_Geniatric
Oct 17, 2023
Merged

[release/8.0] Don't throw for non-generic collection types#32059
ajcvickers merged 1 commit intorelease/8.0from
231016_Geniatric

Conversation

@ajcvickers
Copy link
Copy Markdown
Contributor

@ajcvickers ajcvickers commented Oct 16, 2023

Fixes #32055

Description

We added validation for primitive collection types in RC2, but the validation is throwing for valid collection types if the type is not a generic type.

Customer impact

App that worked on RC1 fails on RC2.

How found

Writing docs.

Regression

Yes, from RC1.

Testing

Added test.

Risk

Low; trivial fix.

@ajcvickers ajcvickers requested a review from a team October 16, 2023 12:16
@ajcvickers ajcvickers added this to the 8.0.0 milestone Oct 16, 2023
@ajcvickers ajcvickers merged commit 87a58cc into release/8.0 Oct 17, 2023
@ajcvickers ajcvickers deleted the 231016_Geniatric branch October 17, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Model validation throws for non-generic collection properties

3 participants