Skip to content

feat: added URL param name to allow setting workspace name #7528

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

Merged
merged 2 commits into from
May 16, 2023

Conversation

MrPeacockNLB
Copy link
Contributor

Description

This is a draft feature request open for RFC. This PR enables user to set the workspace name field in the CreateWorkspacePageView.

I already talked @BrunoQuaresma about this topic. This is currently untested! But @BrunoQuaresma had look and thinks this is fine.

UseCase

We want to create a workspace from a GitHub repository. We've created an Chrome extension to create a link to open the CreateWorkspacePage. We already pass GitHub owner and GitHub repository to the template via URL parameter. This new parameter name should be filled too.

Follow up

As follow up feature we want add the ability to extend the given workspace name via URL param with a random suffix. This has to be enabled as the default is off. This enables to open multiple workspace instances from the repository at the same time as all workspace names are uniqe. Last but not least we want a feature to skip the confirmation on the CreateWorkspacePage and jump directly to the workspace page.

@github-actions
Copy link

github-actions bot commented May 15, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@BrunoQuaresma BrunoQuaresma self-requested a review May 16, 2023 12:46
@MrPeacockNLB
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@MrPeacockNLB MrPeacockNLB force-pushed the feature-set-workspace-name branch from 734961a to a6a6683 Compare May 16, 2023 13:29
cdrcommunity added a commit to coder/cla that referenced this pull request May 16, 2023
@MrPeacockNLB MrPeacockNLB force-pushed the feature-set-workspace-name branch from 53bdd1f to 4419850 Compare May 16, 2023 14:52
@BrunoQuaresma BrunoQuaresma merged commit ad9a29d into coder:main May 16, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 16, 2023
@MrPeacockNLB MrPeacockNLB deleted the feature-set-workspace-name branch May 16, 2023 17:10
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.

2 participants