-
Notifications
You must be signed in to change notification settings - Fork 95
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
Add a useDownload hook #4069
Add a useDownload hook #4069
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. OpenSSF Scorecard
Scanned Manifest Files |
Conventional Commits Report
🚀 Conventional commits found. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4069 +/- ##
==========================================
- Coverage 72.44% 72.26% -0.18%
==========================================
Files 1066 1066
Lines 121953 121747 -206
Branches 6047 6014 -33
==========================================
- Hits 88351 87985 -366
- Misses 33571 33731 +160
Partials 31 31 ☔ View full report in Codecov by Sentry. |
The useDownload hook in conjunction with the Download component should replace the withDownload HOC in future.
Combine Download with useDownload in a simple test.
77b05c4
to
b3f679b
Compare
What
Add a useDownload hook
Why
The useDownload hook in conjunction with the Download component should replace the withDownload HOC in future.
References
HOCless enities pages
Checklist