-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Support multiple names in JsonSubType.Type
#2761
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
Comments
link to PR : #2762 |
PR looks good; added note on #2762 wrt CLA. |
Implement #2761 (support multiple type names for `@JsonSubtypes.Type`)
Implement #2761 (support multiple type names for `@JsonSubtypes.Type`)
@swayamraina I finally got this all merged: thank you very much for this contribution! It will be included in 2.12.0 release. |
@cowtowncoder should I also raise an issue to remove the singular version of this field to be removed in |
I thought about that, and it would otherwise make sense but I am planning to keep 2.x and 3.x annotations compatible -- so that unlike all other components, you can actually use either one. So at this point I think this sub-optimality can live on. Thank you for suggesting it though, it is good to follow up properly with clean up, keeping codebase as tidy as possible. |
sure, this also makes sense. |
This issue is linked to FasterXML/jackson-annotations#171
The text was updated successfully, but these errors were encountered: