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

added Netlify Managed DNS endpoints #207

Closed
wants to merge 1 commit into from
Closed

added Netlify Managed DNS endpoints #207

wants to merge 1 commit into from

Conversation

oscartbeaumont
Copy link

@oscartbeaumont oscartbeaumont commented Dec 22, 2019

PR #163 has not had the requested changes added for a long time and I would like the API endpoint for a project I am working on which is being held up by this. This PR contains those requested changes and is based off the latest master branch.

I used @futuregerald's PR and modified the requested changes in so it can be merged.

Note: As per the contributing guidelines I have not regenerated the Go Client until a reviewer is happy with the changes to the swagger.yml file. By the looks of it, this (and a weird TLS error) caused Travis to fail.

I used @futuregerald's PR and modified the requested changes in so it can be merged.

Co-Authored-By: Gerald Onyango <futuregerald@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Dec 22, 2019

Deploy preview for open-api failed.

Built with commit 81108bf

https://app.netlify.com/sites/open-api/deploys/5dfeff0e90d67400087e7a1c

@keiko713
Copy link
Contributor

keiko713 commented May 8, 2020

I'm going to close this as we merged #211
Sorry that this PR didn't make it but thanks for the contribution!

@keiko713 keiko713 closed this May 8, 2020
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.

None yet

2 participants