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

Mistral: adding workflow executions endpoint #1043

Merged
merged 2 commits into from
Jun 19, 2017

Conversation

rakhmerov
Copy link
Contributor

  • Made ActionResponse able to various 2xx HTTP codes, not only 200
    ** Added one more static method ActionResponse.actionSuccess() taking
    an integer code
    ** Fixed deletion tests for workflow, workbook and actions definitions
    to expect 204 code. 200 was expected due to aforementioned limitation
  • Added an endpoint for Mistral workflow executions

rakhmerov added 2 commits June 6, 2017 17:24
* Added one more static method ActionResponse.actionSuccess() taking
  an integer code
* Fixed deletion tests for workflow, workbook and actions definitions
  to expect 204 code. 200 was expected due to aforementioned limitation
@rakhmerov
Copy link
Contributor Author

@vinodborole, @auhlig: Please review this PR when you have time. Thanks.

@vinodborole
Copy link
Contributor

LGTM

@rakhmerov
Copy link
Contributor Author

ok

@rakhmerov
Copy link
Contributor Author

@auhlig Hi, can you please review it?

@vinodborole vinodborole merged commit 8cfb087 into ContainX:master Jun 19, 2017
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