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

Update the provider's documentation regarding the apikey and secretkey #42

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

Conversation

plemelin
Copy link

@plemelin plemelin commented Nov 3, 2023

The documentation (both on https://github.com/DNSMadeEasy/terraform-provider-dme and https://registry.terraform.io/providers/DNSMadeEasy/dme/latest/docs) does not provide the information regarding the DME_API_KEY and DME_SECRET_KEY. As with most providers, secrets can be provided with the environment variables but it is not documented anywhere.

Additionally, if you search the closed issues, you can see outdated mentions to apikey and secretkey in this issue. The new variables names were added with this PR

Finally there was a broken image link (I used the wayback machine to see what it looked like in the past) and I used the logos from the HashiCorp Brand website. Some minor whitespace fixes are in there too.

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.

1 participant