Skip to content

fix(sdk): let a select or union render with no option preselected #3871

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
dr-bonez merged 1 commit into from
Sep 2, 2026
Merged

fix(sdk): let a select or union render with no option preselected #3871

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions projects/start-sdk/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,10 @@
one canonical origin. See
[Nominating an Address to Open](https://docs.start9.com/packaging/interfaces.html#nominating-an-address-to-open).

- **`Value.select`, `Value.dynamicSelect`, `Value.union` and `Value.dynamicUnion`
accept `default: null`**, which renders the field unselected and holds the form
unsubmittable until the user picks one

### Fixed

- **Scaffolded package CI builds a draft PR when it becomes ready and rebuilds
Expand Down
19 changes: 10 additions & 9 deletions shared-libs/ts-modules/start-core/lib/actions/input/builder/value.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -885,12 +885,12 @@ export class Value<
/** Supplementary text rendered persistently beneath the field. */
footnote?: string | null
/**
* @description Determines if the field is required. If so, optionally provide a default value from the list of values.
* @description The option to preselect. `null` leaves the field unselected; a selection is required either way.
* @type { (keyof Values & string) | null }
* @example default: null
* @example default: 'radio1'
*/
default: keyof Values & string
default: (keyof Values & string) | null
/**
* @description A mapping of unique radio options to their human readable display format.
* @example
Expand Down Expand Up @@ -937,7 +937,7 @@ export class Value<
description?: string | null
warning?: string | null
footnote?: string | null
default: string
default: string | null
values: Values
disabled?: false | string | string[]
},
Expand Down Expand Up @@ -1223,11 +1223,12 @@ export class Value<
warning?: string | null
variants: Variants<VariantValues>
/**
* @description Provide a default value from the list of variants.
* @type { string }
* @description The variant to preselect. `null` leaves the field unselected; a selection is required either way.
* @type { (keyof VariantValues & string) | null }
* @example default: null
* @example default: 'variant1'
*/
default: keyof VariantValues & string
default: (keyof VariantValues & string) | null
/**
* @description Once set, the value can never be changed.
* @default false
Expand Down Expand Up @@ -1269,7 +1270,7 @@ export class Value<
description?: string | null
warning?: string | null
variants: Variants<VariantValues>
default: keyof VariantValues & string
default: (keyof VariantValues & string) | null
disabled: string[] | false | string
},
OuterType
Expand All @@ -1292,7 +1293,7 @@ export class Value<
description?: string | null
warning?: string | null
variants: Variants<VariantValues>
default: keyof VariantValues & string
default: (keyof VariantValues & string) | null
disabled: string[] | false | string
},
OuterType
Expand All @@ -1318,7 +1319,7 @@ export class Value<
description?: string | null
warning?: string | null
variants: Variants<VariantValues>
default: keyof VariantValues & string
default: (keyof VariantValues & string) | null
disabled: string[] | false | string
},
OuterType
Expand Down