-
-
Notifications
You must be signed in to change notification settings - Fork 581
feat(react-form): add useTypedAppFormContext
#1826
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: c32a21f The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
useTypedAppFormContextuseTypedAppFormContext
|
View your CI Pipeline Execution ↗ for commit c32a21f
☁️ Nx Cloud last updated this comment at |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1826 +/- ##
===========================================
- Coverage 90.35% 55.12% -35.23%
===========================================
Files 38 18 -20
Lines 1752 234 -1518
Branches 444 34 -410
===========================================
- Hits 1583 129 -1454
+ Misses 149 93 -56
+ Partials 20 12 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Just wondering when we can expect this to be released? |
|
In the next few days! I let this one stay for too long. I'll write the documentation and add it in. |
28d7e28 to
754b76b
Compare
🎯 Changes
This PR adds the
useTypedAppFormContexthook, which accepts aformOptionsfor typing, while using context to get the form instance instead.This is a less type safe variant of
withForm, which is why we will only recommend it in cases where passing as prop is not possible.🚀 Release Impact
Todos