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 AllowMemberTokenManagement to Team #922

Merged

Conversation

juliannatetreault
Copy link
Contributor

@juliannatetreault juliannatetreault commented Jun 25, 2024

Description

This PR adds a new team permission - AllowMemberTokenManagement - to the Team block. AllowMemberTokenManagement is true by default and can only be managed by Owners or users with "Manage Teams" permissions.

Additionally, this PR adds tests for the new permission.

Testing plan

  1. AllowMemberTokenManagement is true by default when Teams are created.
  2. AllowMemberTokenManagement is read for Teams.
  3. AllowMemberTokenManagement is updatable for Teams.

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-16199-add-team-token-management branch from a7e2b3f to bbcc300 Compare June 25, 2024 18:53
CHANGELOG.md Outdated Show resolved Hide resolved
team.go Outdated Show resolved Hide resolved
team.go Outdated Show resolved Hide resolved
team.go Outdated Show resolved Hide resolved
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-16199-add-team-token-management branch 2 times, most recently from 598e311 to 00fda13 Compare July 2, 2024 16:06
team.go Show resolved Hide resolved
JarrettSpiker
JarrettSpiker previously approved these changes Jul 15, 2024
emlanctot
emlanctot previously approved these changes Jul 15, 2024
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-16199-add-team-token-management branch from c6e08b7 to 19b3a1a Compare July 30, 2024 19:45
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-16199-add-team-token-management branch from 19b3a1a to 037fcab Compare July 30, 2024 19:47
@juliannatetreault juliannatetreault marked this pull request as ready for review July 30, 2024 20:27
@juliannatetreault juliannatetreault force-pushed the juliannatetreault/TF-16199-add-team-token-management branch from 635287f to 9c996d7 Compare July 30, 2024 21:05
Copy link
Contributor

@emlanctot emlanctot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 🛹 ✨

@juliannatetreault juliannatetreault merged commit 517bca1 into main Jul 31, 2024
7 checks passed
@juliannatetreault juliannatetreault deleted the juliannatetreault/TF-16199-add-team-token-management branch July 31, 2024 17:12
Copy link

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

4 participants