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

Add CreateRecord/RemoveRecord methods #883

Closed
wants to merge 8 commits into from

Conversation

mdbraber
Copy link
Contributor

@mdbraber mdbraber commented May 7, 2019

Example of how to implement CreateRecord/RemoveRecord as per discussion in #720

@mholt
Copy link
Contributor

mholt commented Dec 1, 2019

I actually think this would be really useful.

@mdbraber
Copy link
Contributor Author

mdbraber commented Dec 1, 2019

I actually think this would be really useful.

Thanks for chiming in. I was back working at this a bit and this works fine for my own provider (TransIP) and is easy to change in the current code. If there's any way I can help to move this along I'd be happy to support!

@ldez
Copy link
Member

ldez commented Sep 30, 2020

I'm going to close this PR because at the moment I'm still thinking it's not the right abstraction for the challenges.
We can continue to discuss in the related issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants