Skip to content

Document deploy cancel #217

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

Merged
merged 4 commits into from
Feb 25, 2020
Merged

Document deploy cancel #217

merged 4 commits into from
Feb 25, 2020

Conversation

vbrown608
Copy link
Contributor

Documents the endpoint to cancel an in-progress build/depoy.

@vbrown608 vbrown608 requested a review from ehmicky February 25, 2020 18:24
@netlify
Copy link

netlify bot commented Feb 25, 2020

Deploy preview for open-api ready!

Built with commit aa9051d

https://deploy-preview-217--open-api.netlify.com

Copy link
Contributor

@ehmicky ehmicky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The OpenAPI definition changes look good! Thank you so much for the speed on this!

There are some new files (swagger_flat.json and swagger_go.json). It looks like those are added by the generate.go file. I am not too familiar with the Go setup, but based on the CI tests, I think there might be an issue there.

@ehmicky ehmicky self-requested a review February 25, 2020 19:19
@vbrown608 vbrown608 merged commit d74b303 into master Feb 25, 2020
@vbrown608 vbrown608 deleted the document-deploy-cancel branch February 25, 2020 19:25
@@ -628,6 +628,26 @@ paths:
$ref: '#/definitions/deploy'
default:
$ref: '#/responses/error'
/sites/{site_id}/deploys/{deploy_id}/cancel:
Copy link
Contributor

@kitop kitop Mar 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Late to the party, but URL for this is /deploys/{deploy_id}/cancel. with /sites/{site_id} will give a 404 :(

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.

3 participants