8000 fix: show preset parameters when they have diagnostics (#18258) · coder/coder@f46810a · GitHub
[go: up one dir, main page]

Skip to content

Commit f46810a

Browse files
blink-so[bot]jaaydenh
authored andcommitted
fix: show preset parameters when they have diagnostics (#18258)
Always show preset parameters in CreateWorkspacePageViewExperimental if the preset parameter has any diagnostics, regardless of the showPresetParameters toggle state. This ensures that users can see and address errors in preset parameters even when the "Show preset parameters" toggle is disabled. Fixes coder/internal#651 --------- Co-authored-by: blink-so[bot] <211532188+blink-so[bot]@users.noreply.github.com> Co-authored-by: Jaayden Halko <jaayden@coder.com>
1 parent b7a0476 commit f46810a

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

site/src/pages/CreateWorkspacePage/CreateWorkspacePageViewExperimental.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -620,8 +620,12 @@ export const CreateWorkspacePageViewExperimental: FC<
620620
creatingWorkspace ||
621621
isPresetParameter;
622622

623-
// Hide preset parameters if showPresetParameters is false
624-
if (!showPresetParameters && isPresetParameter) {
623+
// Always show preset parameters if they have any diagnostics
624+
if (
625+
!showPresetParameters &&
626+
isPresetParameter &&
627+
parameter.diagnostics.length === 0
628+
) {
625629
return null;
626630
}
627631

0 commit comments

Comments
 (0)
0