Skip to content

feat(site/src/pages/TemplateBuilder): add provisioner warning to Template Builder form - #26935

Merged
jeremyruppel merged 1 commit into
mainfrom
feat/template-builder-provisioner-warning
Jul 1, 2026
Merged

feat(site/src/pages/TemplateBuilder): add provisioner warning to Template Builder form#26935
jeremyruppel merged 1 commit into
mainfrom
feat/template-builder-provisioner-warning

Conversation

@jeremyruppel

@jeremyruppel jeremyruppel commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Show a warning in the Template Builder customizations step when the selected organization has no provisioner daemons connected. This matches the existing behavior in the current CreateTemplateForm.

Changes

  • Added a provisionerDaemons query in TemplateCustomizationsStep, gated on the selected org
  • Renders an Alert warning with a link to provisioner docs when no daemons are found
  • Warning is informational only and does not block form submission
Implementation plan

The current CreateTemplateForm (used for upload/starter/duplicate flows) shows a warning when the selected organization has no provisioner daemons connected. The new Template Builder wizard (/templates/new/builder) had an org picker in TemplateCustomizationsStep but did not perform this check.

What was done

  1. Imported provisionerDaemons query helper, Alert, Link, and docs utility into TemplateCustomizationsStep.tsx
  2. Added a useQuery call for provisioner daemons, enabled only when an org is selected
  3. Computed showProvisionerWarning (true when the provisioners list is empty)
  4. Added a local ProvisionerWarning component rendering the same warning text and docs link as CreateTemplateForm
  5. Rendered the warning below the OrganizationAutocomplete picker

Design decisions

  • Warning is informational only (does not block submission), matching the existing form behavior and the TODO rationale: a user may connect a provisioner without refreshing
  • Used the project's Link component (#/components/Link/Link) instead of MUI Link
  • Kept ProvisionerWarning as a local component rather than extracting to shared, since the component is small (~8 lines)

🤖 Generated by Coder Agents on behalf of @jeremyruppel

Screenshot 2026-07-01 at 3 56 58 PM

@jeremyruppel jeremyruppel changed the title feat(site/TemplateBuilder): add provisioner warning to Template Builder form feat(site/src/pages/TemplateBuilder): add provisioner warning to Template Builder form Jul 1, 2026
…er form

Show a warning in the Template Builder customizations step when the
selected organization has no provisioner daemons connected. This matches
the existing behavior in CreateTemplateForm.
@jeremyruppel
jeremyruppel force-pushed the feat/template-builder-provisioner-warning branch from c4f872c to c73d3ff Compare July 1, 2026 19:59
@jeremyruppel
jeremyruppel marked this pull request as ready for review July 1, 2026 19:59
@jeremyruppel
jeremyruppel requested a review from aqandrew July 1, 2026 19:59
@jeremyruppel
jeremyruppel merged commit 1101a0f into main Jul 1, 2026
35 of 36 checks passed
@jeremyruppel
jeremyruppel deleted the feat/template-builder-provisioner-warning branch July 1, 2026 20:27
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 1, 2026
@jeremyruppel jeremyruppel added cherry-pick/v2.35 Cherry-pick PR targeting release/2.35 cherry-pick and removed cherry-pick/v2.35 Cherry-pick PR targeting release/2.35 labels Jul 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants