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

Add Manage Permissions to Team Organization Access #874

Merged

Conversation

juliannatetreault
Copy link
Contributor

@juliannatetreault juliannatetreault commented Mar 21, 2024

Description

This PR adds new manage permissions - ManageTeams, ManageOrganizationAccess, and AccessSecretTeams - to team organization access. Additionally, this PR adds tests for the new permissions.

Testing plan

  • ManageTeams, ManageOrganizationAccess, and AccessSecretTeams are false by default when Teams are created.
  • ManageTeams, ManageOrganizationAccess, and AccessSecretTeams is read for Teams.
  • ManageTeams, ManageOrganizationAccess, and AccessSecretTeams should be updatable through the OrganizationAccessOptions.

External links

Output from tests

Including output from tests may require access to a TFE instance. Ignore this section if you have no environment to test against.

$ TFE_ADDRESS="https://example" TFE_TOKEN="example" go test ./... -v -run TestFunctionsAffectedByChange

...

@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch from 68d2220 to d62efb0 Compare March 25, 2024 17:21
@juliannatetreault juliannatetreault requested a review from a team April 17, 2024 13:41
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch from d62efb0 to 05f4417 Compare April 17, 2024 13:48
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch from 05f4417 to c9ca2fb Compare April 24, 2024 15:35
CHANGELOG.md Outdated Show resolved Hide resolved
team.go Outdated Show resolved Hide resolved
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch from c9ca2fb to f6f97eb Compare April 24, 2024 16:21
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch 2 times, most recently from cca8222 to a0d8703 Compare April 30, 2024 17:29
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-14028-add-manage-permissions branch from 667a46e to 86d24e4 Compare May 1, 2024 13:54
@juliannatetreault juliannatetreault marked this pull request as ready for review May 1, 2024 14:22
@juliannatetreault juliannatetreault requested a review from a team as a code owner May 1, 2024 14:22
@juliannatetreault juliannatetreault merged commit bc87a54 into main May 1, 2024
8 checks passed
@juliannatetreault juliannatetreault deleted the juliannatetreault/TF-14028-add-manage-permissions branch May 1, 2024 15:56
Copy link

github-actions bot commented May 1, 2024

Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes.

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

2 participants