Skip to content
This repository was archived by the owner on Mar 8, 2022. It is now read-only.

Remove auth0_role.role_id #202

Merged
merged 2 commits into from
Apr 14, 2020
Merged

Remove auth0_role.role_id #202

merged 2 commits into from
Apr 14, 2020

Conversation

alexkappa
Copy link
Owner

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Fixes #193

Changes proposed in this pull request:

  • auth0_role: remove role_id field.

Output from acceptance testing:

$ make testacc TESTS=TestAccRole                                    [14:05:33]
==> Checking that code complies with gofmt requirements...
?       github.com/terraform-providers/terraform-provider-auth0 [no test files]
=== RUN   TestAccRole
--- PASS: TestAccRole (4.75s)
PASS
coverage: 15.1% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0   4.798s  coverage: 15.1% of statements
?       github.com/terraform-providers/terraform-provider-auth0/auth0/internal/debug    [no test files]
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0/internal/random   0.048s  coverage: 0.0% of statements [no t
ests to run]
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0/internal/validation       0.015s  coverage: 0.0% of statements [no tests to run]
?       github.com/terraform-providers/terraform-provider-auth0/version [no test files]

@alexkappa alexkappa merged commit 379335f into master Apr 14, 2020
@yvovandoorn yvovandoorn deleted the fix-role-id branch March 24, 2021 14:29
nikolap added a commit to nikolap/terraform-provider-auth0 that referenced this pull request Jul 14, 2021
@nikolap
Remove role_id from role documentation
aa6954a 
See alexkappa#202
@nikolap nikolap mentioned this pull request Jul 14, 2021
Merged
sergiught pushed a commit to nikolap/terraform-provider-auth0 that referenced this pull request Jan 26, 2022
@nikolap @sergiught
Remove role_id from role documentation
9bd768c 
See alexkappa#202
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
resource/auth0_role
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Setting rule_id on an auth0_rule throws a 400
1 participant
@alexkappa