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

chore: useSetFormValues TS Support #4475

Merged
merged 1 commit into from
Sep 15, 2023
Merged

chore: useSetFormValues TS Support #4475

merged 1 commit into from
Sep 15, 2023

Conversation

ivan-angjelkoski
Copy link
Contributor

🔎 Overview

This PR Adds Typescript support for the useSetFormValues hook.


type Form = {
username:string
}

const setFormValues = useSetFormValues<Form>()

setFormValues({
username: 'johndoe' // we get TS autocomplete here
})

Issues affected

@codecov
Copy link

codecov bot commented Sep 15, 2023

Codecov Report

Merging #4475 (efdc25d) into main (a023c70) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #4475   +/-   ##
=======================================
  Coverage   98.15%   98.15%           
=======================================
  Files          79       79           
  Lines        6226     6226           
  Branches     1179     1179           
=======================================
  Hits         6111     6111           
  Misses        115      115           
Files Changed Coverage Δ
packages/vee-validate/src/useSetFormValues.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@logaretm logaretm merged commit e1a4b23 into logaretm:main Sep 15, 2023
8 checks passed
@logaretm
Copy link
Owner

Nice one, thanks!

@ivan-angjelkoski
Copy link
Contributor Author

Thank You! can you republish the package since i need to install it in my project? :D

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.

None yet

2 participants