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

fix: use team id instead of slug #205

Merged
merged 1 commit into from
Jan 22, 2025
Merged

fix: use team id instead of slug #205

merged 1 commit into from
Jan 22, 2025

Conversation

matijs
Copy link
Member

@matijs matijs commented Jan 22, 2025

For github_repository_collaborators team blocks, use the id instead of the slug

@matijs matijs requested a review from a team as a code owner January 22, 2025 09:12
@matijs matijs force-pushed the fix/team-slug-vs-id branch 4 times, most recently from 7f63f72 to 7609781 Compare January 22, 2025 10:40
@matijs matijs enabled auto-merge (rebase) January 22, 2025 10:50
@matijs matijs requested a review from Robbert January 22, 2025 10:51
For `github_repository_collaborators` `team` blocks, use the team id
instead of the team slug.

[This change][1] in the GitHub integration caused a lot of noise that is
now fixed.

[1]: integrations/terraform-provider-github#2420
@matijs matijs force-pushed the fix/team-slug-vs-id branch from 7609781 to 83b29b6 Compare January 22, 2025 10:57
@matijs matijs merged commit fc2fd1e into main Jan 22, 2025
3 checks passed
@matijs matijs deleted the fix/team-slug-vs-id branch January 22, 2025 11:04
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.

2 participants