-
Notifications
You must be signed in to change notification settings - Fork 220
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
feat: add readFormData
util
#421
Conversation
Codecov Report
@@ Coverage Diff @@
## main #421 +/- ##
==========================================
+ Coverage 78.23% 78.36% +0.13%
==========================================
Files 26 26
Lines 2775 2792 +17
Branches 407 407
==========================================
+ Hits 2171 2188 +17
Misses 604 604
|
Update: Checking MDN docs, it seems we cannot directly construct |
readFormData
and eventToRequest
@pi0 Updated the method names to |
Hi @Hebilicious, |
@neo-franklin-tina Are you talking about multipart/form-data ? It should work by default yes. |
5be88e0
to
81ad4fc
Compare
Yes, something like that. |
readFormData
and eventToRequest
readFormData
util
Rebased util with |
Resolves #419
π Linked issue
β Type of change
π Description
This PR adds support for Request and FormData in event handler.
2 new helpers are introduced :
getRequestFromEvent
getFormData
This helpers relies on Request and FormData to be available in the global scope as per the Fetch specification, which means the tests are disabled for node < 18 (which is fine as 18 reaches EOL in 3 months)
π Checklist