Skip to content
This repository was archived by the owner on Jul 5, 2023. It is now read-only.

Variable Set Permission Update #2238

Merged
merged 5 commits into from
Apr 21, 2022
Merged

Conversation

JPurcellHCP
Copy link
Contributor

Added info about manage workspaces being tied to the ability to change Variable Sets.

@vercel
Copy link

vercel bot commented Apr 13, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hashicorp/terraform-website/7UhZ1igBMUQ6Sk3yS8N1Q5dFDZR8
✅ Preview: https://terraform-website-git-jp-var-set-permission-update-hashicorp.vercel.app

@hashicorp-cla
Copy link

hashicorp-cla commented Apr 13, 2022

CLA assistant check
All committers have signed the CLA.

Copy link

@jaredfholgate jaredfholgate left a comment

Choose a reason for hiding this comment

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

Looks good, consider adding a note to the page which documents variable sets too in order to make it as easy as possible to find this info.

Copy link

@jaredfholgate jaredfholgate left a comment

Choose a reason for hiding this comment

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

Look great, thanks.

Copy link
Contributor

@laurapacilio laurapacilio left a comment

Choose a reason for hiding this comment

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

Approving so I don't block this work, but i had two wording suggestions for you :-). Thank you!

@vercel
Copy link

vercel bot commented Apr 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
terraform-website ❌ Failed (Inspect) Apr 21, 2022 at 8:43AM (UTC)

@JPurcellHCP JPurcellHCP merged commit ec7a22c into master Apr 21, 2022
@JPurcellHCP JPurcellHCP deleted the jp-var-set-permission-update branch April 21, 2022 08:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants