Skip to content

Conversation

tankosinn
Copy link
Contributor

@tankosinn tankosinn commented May 19, 2025

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

  • Introduce the FormData<S, T> conditional type helper.
  • Update the onSubmit prop and submit event payload to use FormData<S, T>.
  • Add a T generic to the internal _validate function and to Form.validate() to respect the transform option when inferring the form schema.
  • Use the S (FormSchema) generic for Form instead of I (InferInput<S>) enabling schema inference via FormData<S, T>.
  • No runtime behavior changes; all changes are type-only.

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

Copy link

pkg-pr-new bot commented May 19, 2025

npm i https://pkg.pr.new/@nuxt/ui@4188

commit: 45383a8

Copy link
Member

@romhml romhml left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tankosinn :)

@benjamincanac benjamincanac merged commit 37abcc6 into nuxt:v3 May 30, 2025
6 checks passed
@tankosinn tankosinn deleted the fix(Form)/conditionally-typed-form-data branch May 30, 2025 15:00
@benjamincanac benjamincanac added the v3 #1289 label Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3 #1289
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants