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

Python discriminator support #1256

Closed
8 tasks
baywet opened this issue Feb 23, 2022 · 0 comments · Fixed by #1500 or #1200
Closed
8 tasks

Python discriminator support #1256

baywet opened this issue Feb 23, 2022 · 0 comments · Fixed by #1500 or #1200
Assignees
Labels
enhancement New feature or request Python WIP

Comments

@baywet
Copy link
Member

baywet commented Feb 23, 2022

reflection of #648 for Python
depends on #1174

TODO:

  • introduce a new ParsableFactory type in the abstractions serialization package
  • update parse node interface to take the new factory as parameter (getObjectValue, getCollectionOfObjectValues)
  • update deserializer method generation to pass the factory for the properties + update unit tests
  • add factory to request adapter interface and implementation as parameter, and pass the factory to the parse node methods
  • update request executor generation to pass the factory to the request adapter send method + update unit tests
  • bump abstractions, http and serialization packages
  • implement factory method generation + unit tests
  • use refiner AddDiscriminatorMappingsUsingsToParentClasses to add reference to types in discriminator mapping
@baywet baywet added enhancement New feature or request Python labels Feb 23, 2022
@baywet baywet added this to the Community Preview milestone Feb 23, 2022
@samwelkanda samwelkanda linked a pull request Apr 14, 2022 that will close this issue
4 tasks
@ghost ghost added the WIP label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Python WIP
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants