Skip to content

Conversation

briancain
Copy link
Contributor

Description

Prior to this commit, the organization permissions struct left out a couple of permissions around the no-code provisioning feature:

This commit includes those permissions, which are already being returned by the TF API, into the OrganizationPermissions struct.

Fixes WAYP-2963 TF-19913

Testing plan

External links

Output from tests

Prior to this commit, the organization permissions struct left out a
couple of permissions around the no-code provisioning feature:

- https://developer.hashicorp.com/terraform/tutorials/cloud/no-code-provisioning

This commit includes those permissions, which are already being returned
by the TF API, into the `OrganizationPermissions` struct.

Fixes WAYP-2963 TF-19913
@briancain briancain requested a review from haydengomes August 27, 2024 17:44
CanDestroy bool `jsonapi:"attr,can-destroy"`
CanManageNoCodeModules bool `jsonapi:"attr,can-manage-no-code-modules"`

Choose a reason for hiding this comment

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

These are exposed through the organization serializer

haydengomes
haydengomes previously approved these changes Aug 27, 2024
Uk1288
Uk1288 previously approved these changes Aug 27, 2024
Copy link
Contributor

@Uk1288 Uk1288 left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@briancain briancain merged commit 8e308df into main Aug 27, 2024
7 checks passed
@briancain briancain deleted the org/add-no-code-permissions branch August 27, 2024 21:23
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.

3 participants