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 Lexicon provider #546

Closed
wants to merge 4 commits into from
Closed

Add Lexicon provider #546

wants to merge 4 commits into from

Conversation

@doddo
Copy link
Contributor

doddo commented May 17, 2020

Add Lexicon provider per #534 which could potentially solve gandi
support #476, GoDaddy support #365 and maybe Aliyun(WanWang) #386 to name a few
by using this "wrapper" for Lexicon (which has support for these)

All record types possible are not added yet, and although test coverage is 100%, I have yet to assert enough data is correct between the various transformations, but I figure I'd open this PR up for review, and will complete the rest meanwhile, as they'd not introduce any significant changes.

Another caveat is that I have only tried this one with the gandi Lexicon provider.

doddo added 2 commits May 17, 2020
Add Lexicon provider per #534 which could potentially solve gandi
support #476, GoDaddy support #365 and maybe Aliyun(WanWang) #386 to name a few
by using this "wrapper" for Lexicon (which has support for these)
@doddo doddo marked this pull request as draft May 19, 2020
Handle updates of multi value records better.
@doddo doddo force-pushed the doddo:LexiconProvider branch from 9ed5173 to 2bd44e2 May 23, 2020
@doddo doddo marked this pull request as ready for review May 23, 2020
@doddo
Copy link
Contributor Author

doddo commented Jun 3, 2020

I put this one into third party library instead. It can be found here: https://github.com/doddo/octodns-lexicon

@doddo doddo closed this Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.