-
Notifications
You must be signed in to change notification settings - Fork 157
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 sso_team_id to the tfe_team resource and data source #457
Conversation
if v, ok := d.GetOk("sso_team_id"); ok { | ||
options.SSOTeamID = tfe.String(v.(string)) | ||
} else { | ||
options.SSOTeamID = tfe.String("") |
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.
This is the else case you mentioned that can be removed once this PR is through right? (This is more just out of my own curiosity / to reload context and not at all blocking).
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.
yah, that is right
website/docs/r/team.html.markdown
Outdated
@@ -41,6 +41,7 @@ The following arguments are supported: | |||
* `organization` - (Required) Name of the organization. | |||
* `visibility` - (Optional) The visibility of the team ("secret" or "organization"). Defaults to "secret". | |||
* `organization_access` - (Optional) Settings for the team's [organization access](https://www.terraform.io/docs/cloud/users-teams-organizations/permissions.html#organization-level-permissions). | |||
* `sso_team_id` - (Optional) Unique Identifier to control team membership via SAML. Defaults to `null` |
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.
It looks like we link to the docs for the organization_access
attribute above. Should we be doing the same for the SSO team ID docs here and in the datasource docs?
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.
Yah, good idea, updated
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.
LGTM 👍 Would probably be good to get someone from the provider team to give the go-ahead here as well though
d98a204
to
e2f5542
Compare
Description
Add sso_team_id to the tfe_team resource and data source
Testing plan
Output from acceptance tests