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
Removed unmapped composite support #2476
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this @YohDeadfall. See small nit and question, otherwise LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
So in v5 there is no ExpandoObject, but user-defined composites can be read as strings....? |
As a struct or class which must be mapped by invoking |
Thanks, I've only seen the FAQ page that the NotSupportedException pointed me to. |
We've got a breaking change note on that BTW. |
Seen that too, it links to the issue mentioned in OP. For someone that didn't know composite type mapping was a thing it did not provide a lot to go on (but I didn't exactly look around much either), that's why I asked here. Anyway, the Enums and composites docs page still contains a section about dynamic composite type reading. Not sure if it's an oversight or if you're keeping it there for the sake of v4. |
@kerams thanks for pointing that out - I'll remove the info on dynamic composite mapping. |
Closes #2403.