Skip to content

fix: Add organization option for workspace sharing#336

Open
angrycub wants to merge 1 commit intomainfrom
fix/add-organization-sharing
Open

fix: Add organization option for workspace sharing#336
angrycub wants to merge 1 commit intomainfrom
fix/add-organization-sharing

Conversation

@angrycub
Copy link
Copy Markdown
Contributor

@angrycub angrycub commented Apr 23, 2026

Add missing Organization option for workspace sharing

@angrycub angrycub self-assigned this Apr 23, 2026
@angrycub angrycub force-pushed the fix/add-organization-sharing branch from fabf9f9 to d11683d Compare April 23, 2026 21:34
@angrycub angrycub requested a review from ethanndickson April 23, 2026 21:38
@angrycub angrycub added go Pull requests that update Go code labels Apr 23, 2026
@angrycub angrycub requested a review from johnstcn April 27, 2026 13:44
Copy link
Copy Markdown
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

Needs some corresponding tests in internal/provider/template_resource_test.go

Optionally, also a new integration test to validate the feature end-to-end.

@angrycub
Copy link
Copy Markdown
Contributor Author

Should I also add a test for authenticated, since it's not tested anywhere?

@johnstcn
Copy link
Copy Markdown
Member

Should I also add a test for authenticated, since it's not tested anywhere?

Dealer's choice! I won't complain either way though 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants