-
Notifications
You must be signed in to change notification settings - Fork 323
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
New data_source keycloak_realm #160
Conversation
CircleCI failure due to #32 |
mkdocs.yml
Outdated
@@ -4,6 +4,9 @@ repo_url: https://github.com/mrparkers/terraform-provider-keycloak | |||
nav: | |||
- Getting Started: index.md | |||
- Data Sources: | |||
- keycloak_openid_client: data_sources/keycloak_openid_client.md | |||
- keycloak_openid_client_authorization_policy: data_sources/keycloak_openid_client_authorization_policy.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this is awkward, but there actually aren't doc files for these data sources you added. I'd appreciate a PR to add these if you're interested 😄
If not, could you remove these dead links?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whops! My mistake, I planned on adding those documents but I read that you were working on it so I deleted the files, but I forgot to delete the links :D
I'll file a separate PR soon, for now I'm deleting the references.
PR Review Co-Authored-By: Michael Parker <michael@parker.gg>
LGTM, thanks for the PR! Sorry again about the flaky tests |
This adds a new datasource for keycloak_realm.
Acceptance test results: