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

Feature/webdocs #46

Merged
merged 7 commits into from
May 17, 2021
Merged

Feature/webdocs #46

merged 7 commits into from
May 17, 2021

Conversation

pezhore
Copy link
Contributor

@pezhore pezhore commented Apr 22, 2021

This is similar to the existing draft PR #21 , however I have gone ahead and filled in each resource/data source with details and references to the official documentation.

This PR also adds a guide section that offers a sample use case that will be visible in the register (similar to the Unifi provider).

While I agree that incorporating the gen into a git hook would be ideal, manually generating the docs for now will help with the registry visibility/usability.

@pezhore
Copy link
Contributor Author

pezhore commented Apr 22, 2021

Not sure why the tests are failing, this PR only changes the markdown docs, any insight from the maintainers?

@pezhore
Copy link
Contributor Author

pezhore commented Apr 23, 2021

Looks like the tests failed because the plugin sdk was outdated @fbreckle can you kick off the tests again?

@FlxPeters
Copy link
Member

@fbreckle This is a related PR on this, but his one is better. #21

@fbreckle fbreckle self-assigned this May 5, 2021
@pezhore
Copy link
Contributor Author

pezhore commented May 7, 2021

Quick note... as each doc includes a sample usage, this would also address #39.

@fbreckle fbreckle merged commit aa99b79 into e-breuninger:master May 17, 2021
@fbreckle
Copy link
Collaborator

tasty: https://registry.terraform.io/providers/e-breuninger/netbox/latest/docs

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

3 participants