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

🐛 Do not jasonify FormData instances #232

Merged
merged 2 commits into from
Jun 1, 2024
Merged

Conversation

am1rb
Copy link
Contributor

@am1rb am1rb commented May 23, 2024

closes #231

@elbywan elbywan changed the base branch from master to dev June 1, 2024 09:52
@elbywan
Copy link
Owner

elbywan commented Jun 1, 2024

Hey @am1rb,

Thanks a bunch for the PR and the bug report! 🙏

There are some tests failing though, I've taken a look locally and I can push a quick fix on your branch if you're up for it.
I just seem to be missing the rights to collaborate on this branch:

To github.com:am1rb/wretch.git
! [remote rejected] am1rb/master -> am1rb/master (permission denied)
error: failed to push some refs to 'github.com:am1rb/wretch.git' 

@am1rb
Copy link
Contributor Author

am1rb commented Jun 1, 2024

hello @elbywan, no worries. I checked the link you shared and saw that the option is already on for this PR:
image

@am1rb
Copy link
Contributor Author

am1rb commented Jun 1, 2024

FYI: I faced the test error and fixed it in my branch. all tests are green now

@coveralls
Copy link

Coverage Status

coverage: 87.394% (+0.07%) from 87.322%
when pulling 71aa327 on am1rb:master
into cd1b822 on elbywan:dev.

@elbywan elbywan merged commit 0619303 into elbywan:dev Jun 1, 2024
10 checks passed
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.

Wretch loses the request body for the PUT, POST, and PATCH methods if it is FormData
3 participants