Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit c690c7b

Browse files
committed
chore: update to use parameter styling type
1 parent 22a6629 commit c690c7b

File tree

3 files changed

+14
-18
lines changed

3 files changed

+14
-18
lines changed

site/src/api/typesGenerated.ts

Lines changed: 8 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

site/src/modules/workspaces/DynamicParameter/DynamicParameter.tsx

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,6 @@ import { type FC, useEffect, useId, useState } from "react";
3636
import type { AutofillBuildParameter } from "utils/richParameters";
3737
import * as Yup from "yup";
3838

39-
export interface ParameterStyling {
40-
placeholder?: string;
41-
label?: string;
42-
disabled?: boolean;
43-
}
44-
4539
export interface DynamicParameterProps {
4640
parameter: PreviewParameter;
4741
onChange: (value: string) => void;
@@ -170,7 +164,6 @@ const ParameterField: FC<ParameterFieldProps> = ({
170164
id,
171165
}) => {
172166
const value = validValue(parameter.value);
173-
const styling = parameter.styling as ParameterStyling;
174167
const [localValue, setLocalValue] = useState(value);
175168

176169
useEffect(() => {
@@ -188,7 +181,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
188181
>
189182
<SelectTrigger>
190183
<SelectValue
191-
placeholder={styling?.placeholder || "Select option"}
184+
placeholder={parameter.styling?.placeholder || "Select option"}
192185
/>
193186
</SelectTrigger>
194187
<SelectContent>
@@ -249,7 +242,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
249242
onChange(JSON.stringify(values));
250243
}}
251244
hidePlaceholderWhenSelected
252-
placeholder={styling?.placeholder || "Select option"}
245+
placeholder={parameter.styling?.placeholder || "Select option"}
253246
emptyIndicator={
254247
<p className="text-center text-md text-content-primary">
255248
No results found
@@ -305,7 +298,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
305298
}}
306299
disabled={disabled}
307300
/>
308-
<Label htmlFor={parameter.name}>{styling?.label}</Label>
301+
<Label htmlFor={parameter.name}>{parameter.styling?.label}</Label>
309302
</div>
310303
);
311304

@@ -342,7 +335,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
342335
target.style.height = `${target.scrollHeight}px`;
343336
}}
344337
disabled={disabled}
345-
placeholder={styling?.placeholder}
338+
placeholder={parameter.styling?.placeholder}
346339
required={parameter.required}
347340
/>
348341
);
@@ -374,7 +367,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
374367
}}
375368
disabled={disabled}
376369
required={parameter.required}
377-
placeholder={styling?.placeholder}
370+
placeholder={parameter.styling?.placeholder}
378371
{...inputProps}
379372
/>
380373
);

site/src/pages/CreateWorkspacePage/CreateWorkspacePageViewExperimental.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import { useDebouncedFunction } from "hooks/debounce";
2323
import { ArrowLeft, CircleAlert, TriangleAlert } from "lucide-react";
2424
import {
2525
DynamicParameter,
26-
type ParameterStyling,
2726
getInitialParameterValues,
2827
useValidationSchemaForDynamicParameters,
2928
} from "modules/workspaces/DynamicParameter/DynamicParameter";
@@ -493,17 +492,15 @@ export const CreateWorkspacePageViewExperimental: FC<
493492

494493
<div className="flex flex-col gap-9">
495494
{parameters.map((parameter, index) => {
496-
const styling = parameter.styling as ParameterStyling;
497495
const parameterField = `rich_parameter_values.${index}`;
498-
const parameterInputName = `${parameterField}.value`;
499496
const isPresetParameter = presetParameterNames.includes(
500497
parameter.name,
501498
);
502499
const isDisabled =
503500
disabledParams?.includes(
504501
parameter.name.toLowerCase().replace(/ /g, "_"),
505502
) ||
506-
styling?.disabled ||
503+
parameter.styling?.disabled ||
507504
creatingWorkspace ||
508505
isPresetParameter;
509506

0 commit comments

Comments
 (0)