Skip to content

Add createSiteBuild endpoint #124

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 1 commit into from
Jan 11, 2019
Merged

Add createSiteBuild endpoint #124

merged 1 commit into from
Jan 11, 2019

Conversation

stefanjudis
Copy link
Contributor

I'm currently building a netlify menubar client and use the netlify JS SDK. I found out that the swagger api definitions are missing the endpoint definition for a POST to sites/$sitesId/builds which is used by the netlify Admin UI to manually trigger builds (which is functionality I'd like to provide in the Netlify menubar app).

I talked to Dennis from support and he told me that the Netlify Admin UI is the source of truth. This PR adds documentation for the endpoint so that it'll be reflected in the JS Sdk and potentially Go SDK. :)

Copy link
Contributor

@calavera calavera left a comment

Choose a reason for hiding this comment

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

LGTM

@bcomnes , @stefanjudis I think we need to generate the code to add this new route, don't we?

@stefanjudis
Copy link
Contributor Author

@calavera hmm, generate the code? I'm not sure I understand. :)

@calavera
Copy link
Contributor

nevermind, if you use the JS version you don't need to generate anything, I'll take care of the Go version.

@calavera calavera merged commit 171b6a8 into netlify:master Jan 11, 2019
@stefanjudis stefanjudis deleted the add-create-builds-endpoint branch January 11, 2019 18:17
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