Skip to content

Conversation

mtojek
Copy link
Member

@mtojek mtojek commented Sep 3, 2025

Fixes: #15798

This PR extends the Template Embed form to include "default workspace name". The logic handling the "name" GET parameter was already implemented a while ago.

Screenshot 2025-09-03 at 13 22 03

@mtojek mtojek self-assigned this Sep 3, 2025
@mtojek mtojek marked this pull request as ready for review September 3, 2025 11:59
Copy link
Member

@Parkreiner Parkreiner left a comment

Choose a reason for hiding this comment

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

The overall approach looks good to me – just had some concerns about some of the finer details

@mtojek mtojek requested a review from Parkreiner September 4, 2025 10:43
@mtojek mtojek force-pushed the embed-workspace-name branch from 108809c to 6b576aa Compare September 4, 2025 10:45
@mtojek mtojek requested a review from aslilac September 5, 2025 09:33
Copy link
Member

@Parkreiner Parkreiner left a comment

Choose a reason for hiding this comment

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

Checked the stories, and everything looks good. Appreciate you adding a new story for the error case

@mtojek mtojek merged commit 5c1a708 into main Sep 8, 2025
31 of 32 checks passed
@mtojek mtojek deleted the embed-workspace-name branch September 8, 2025 11:58
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2025
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.

Add workspace name to "Embed" section of templates UI
3 participants