Skip to content

Option to upload file directly in the request body and set http method. #8

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

manuelstofer
Copy link

Hi

I added a few things that seemed useful to me.

  • Fixes a broken test case. Upload was created without file.
  • Add support for uploading files directly in request body without multipart encoding.
  • Use a blob instead of the file input, makes it more convenient to run the tests. It also fixes a problem that the change event handlers are executed in each following test again.
  • Adds the option to set the http method.

If you prefer i can send the commits as separate pull requests.

@Swatinem
Copy link
Contributor

I have merged #9, so upload.to() now takes an options object. Would be awesome if you could update your patch for that.

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.

2 participants