Skip to content

fix(actions): allow empty nullable enums in action forms - #34

Merged
MathiasReichardt merged 1 commit into
mainfrom
fix/required-regression
Oct 6, 2026
Merged

MathiasReichardt merged 1 commit into
mainfrom
fix/required-regression

Conversation

@MathiasReichardt

Copy link
Copy Markdown
Contributor
  • treat enum containing null as nullable, e.g. {"enum": [..., null]} without type
  • treat oneOf/anyOf with a null branch as nullable
  • skip the non-null check for fields without type, enum or const, e.g. {}
  • mark non-nullable fields as required, so the asterisk shows why Submit is disabled
  • add form level tests that run each schema shape through the schema simplifier
  • regression from Add file upload action parameters #32, which enforced non-null on every field and disabled Submit

- treat enum containing null as nullable, e.g. {"enum": [..., null]} without type
- treat oneOf/anyOf with a null branch as nullable
- skip the non-null check for fields without type, enum or const, e.g. {}
- mark non-nullable fields as required, so the asterisk shows why Submit is
  disabled
- add form level tests that run each schema shape through the schema simplifier
- regression from #32, which enforced non-null on every field and disabled Submit
@MathiasReichardt
MathiasReichardt merged commit c056949 into main Oct 6, 2026
2 checks passed
@MathiasReichardt
MathiasReichardt deleted the fix/required-regression branch October 6, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants