Skip to content

Commit 0a17d51

Browse files
fix: update multi-select to use value prop for dynamic defaults
Fixes issue where multi-select parameters don't update their selected values when dynamic defaults change. Changed from using defaultOptions to value prop in MultiSelectCombobox to ensure proper reactivity. Fixes coder/internal#753 Co-authored-by: jaaydenh <1858163+jaaydenh@users.noreply.github.com>
1 parent aa1a985 commit 0a17d51

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -476,7 +476,7 @@ const ParameterField: FC<ParameterFieldProps> = ({
476476
<MultiSelectCombobox
477477
inputProps={{ id }}
478478
options={options}
479-
defaultOptions={selectedOptions}
479+
value={selectedOptions}
480480
onChange={(newValues) => {
481481
const values = newValues.map((option) => option.value);
482482
onChange(JSON.stringify(values));

0 commit comments

Comments
 (0)