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 records without replacing all existing ones #56

Open
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

anna-kampani
Copy link

@anna-kampani anna-kampani commented Jan 11, 2023

What's the context of this PR?

It gives the ability to select if the resource records will be just added to godaddy or if they will replace existing records.

  • Adds overwrite bool property to record resource:
    If set to true it will replace all records on godaddy with the list or records provided in the resource (existing functionality)
    If set to false it will only add the provided records without affecting any of the existing ones

  • Splits UpdateDomainRecords to two functions, one that replaces records (existing functionality) and one that only adds records.
    Based on the godaddy records API documentation:
    image
    image

  • Implements provider read function to update terraform state after the creation of the new records
    TF docs

  • Adds Terraform docs plugin to automatically generate docs

  • Upgrades go to latest version 1.19

Issues addressed in this PR

@ms-ati
Copy link

ms-ati commented Jun 15, 2023

(@n3integration Canary - is this project alive?)

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