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

Add support for ExportDefinition #844

Merged
merged 4 commits into from
Oct 15, 2024

Conversation

hkad98
Copy link
Contributor

@hkad98 hkad98 commented Oct 15, 2024

No description provided.

JIRA: TRIVIAL
risk: low
@hkad98 hkad98 enabled auto-merge October 15, 2024 11:53
There was an issue resolving oneOf. A workaround was introduced some time ago, but it did not work for ExportDefinitionRequest. ExportDefinitionRequest was correctly matched, but it did not pass the assert, so it failed with the error that none oneOf could be used. Therefore, assert was extended for the use case when all model_kwargs are presented in oneof_class.openapi_types. This new condition and the old one should cover both cases.

JIRA: PSDK-209
risk: low
JIRA: PSDK-209
risk: low
JIRA: PSDK-209
risk: low
@hkad98 hkad98 force-pushed the jkd/export-definition branch from 097f3aa to 68949c8 Compare October 15, 2024 12:01
@hkad98 hkad98 merged commit be0ea8a into gooddata:master Oct 15, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants