Skip to content

Align command validation and currentValues nullish handling#2307

Merged
woksin merged 2 commits into
mainfrom
fix/command-required-string-semantics
Jun 19, 2026
Merged

Align command validation and currentValues nullish handling#2307
woksin merged 2 commits into
mainfrom
fix/command-required-string-semantics

Conversation

@woksin

@woksin woksin commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes two command correctness issues in Arc React forms and dialogs:

  • Empty string values were rejected by core required-property validation even when no explicit empty-string validator existed, making client validation stricter than server/domain validation.
  • Reactive currentValues / useCommand syncing ignored explicit null and undefined values, so cleared external state could leave stale command values for validation and submit.

Fixed

Changed

@github-actions

Copy link
Copy Markdown

NuGet packages for this PR, e.g. Cratis.Arc:
https://github.com/cratis/arc/packages/1655206?version=20.43.7

@woksin woksin added the patch label Jun 19, 2026
@woksin woksin changed the title Align required string validation with nullability Align command validation and currentValues nullish handling Jun 19, 2026
@woksin woksin merged commit 1906ab2 into main Jun 19, 2026
18 checks passed
@woksin woksin deleted the fix/command-required-string-semantics branch June 19, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant