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.

Sign up for GitHub

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

Jump to bottom

?include_attributes regression fix for /v1/executions API endpoint, update openapi.yaml #4226

Merged
merged 10 commits into from
Jul 6, 2018
Merged

?include_attributes regression fix for /v1/executions API endpoint, update openapi.yaml #4226

merged 10 commits into from
Jul 6, 2018

Conversation

Kami
Copy link
Member

@Kami Kami commented Jul 4, 2018

This pull request includes two changes:

  1. It fixes regression with ?include_attributes query param filter in the GET /v1/executions API endpoint
  2. Updates out of sync openapi.yaml file (it was missing some new orchestra related attributes on the Execution object)

@Kami Kami added this to the 2.8.0 milestone Jul 4, 2018
@Kami Kami requested a review from enykeev July 4, 2018 13:06
@Kami Kami mentioned this pull request Jul 5, 2018
Merged
@Kami Kami merged commit c34bbfd into master Jul 6, 2018
@Kami Kami deleted the resposne_fix branch July 6, 2018 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@enykeev enykeev enykeev approved these changes

Assignees
No one assigned
Labels
API bug
Projects
None yet
Milestone
2.8.0
Development

Successfully merging this pull request may close these issues.
2 participants
@Kami @enykeev