Skip to content
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.

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

[2.x] Fix useForm typehinting issue #2273

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

noxasch
Copy link

@noxasch noxasch commented Mar 11, 2025

resolves #2188 which introduce from #2060.

This PR should cover all 3 framework (react, svelte, vue3) issue with typescript.

Update test-app pages for each framework that use useForm to use typescript to validate this.

@JosephusPaye
Copy link

JosephusPaye commented Mar 13, 2025

Thanks for taking a stab at this @noxasch. I just ran into this problem too.

Unfortunately, this PR doesn't seem to solve the problem. I believe the type with the issue is FormDataType, which this PR doesn't change. Here are some TypeScript playground links demonstrating this:

@noxasch noxasch force-pushed the fix-useform-typehinting-issue branch from 7c9ddf0 to d50f5f0 Compare March 13, 2025 06:10
@noxasch noxasch force-pushed the fix-useform-typehinting-issue branch from d50f5f0 to 7844175 Compare March 13, 2025 06:32
@noxasch noxasch force-pushed the fix-useform-typehinting-issue branch from cdfa0cf to d093a5f Compare March 13, 2025 07:55
@noxasch noxasch changed the title fix: fix useForm typehinting issue Fix useForm typehinting issue Mar 13, 2025
@noxasch noxasch changed the title Fix useForm typehinting issue [2.x] Fix useForm typehinting issue Mar 13, 2025
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.0.1] Typehinting useForm with interface gives error.
2 participants