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

0.19 to beta #1

Open
wants to merge 27 commits into
base: beta
Choose a base branch
from
Open

0.19 to beta #1

wants to merge 27 commits into from

Conversation

thomas-33across
Copy link
Owner

Instructions

Please read and follow the instructions before creating and submitting a pull request:

  • Create an issue explaining the feature. It could save you some effort in case we don't consider it should be included in axios.
  • If you're fixing a bug, try to commit the failing test/s and the code fixing it in different commits.
  • Ensure you're following our contributing guide.

⚠️👆 Delete the instructions before submitting the pull request 👆⚠️

Describe your pull request here.

emilyemorehouse and others added 27 commits August 10, 2018 12:36
- Remove code coverage from SauceLabs CI, as it causes tests to hang.
- Update Safari browser tests to handle Safari 9-11 with proper OS
settings
* Add `getUri` signature to TypeScript definition.

This is in support of axios#1624.

* Make configuration optional.
Clarify in README that default timeout is 0 (no timeout)
Resolves axios#1158

This modifies http.js to uppercase the HTTP method, similar to xhr.js, before passing the request off to the transport. This causes follow-redirects to preserve the HTTP method when automatically making a request to the next URL.
This commit fix building url with hash map (fragment identifier) when parameters are present: they must not be added after `#`, because client cut everything after `#`
* Update README.md. Querystring libraries note

* Typo in README.md

Co-Authored-By: airs0urce <airs0urce0@gmail.com>

* Update README.md

Co-Authored-By: airs0urce <airs0urce0@gmail.com>
It seems that `responseType: 'blob'` doesn't actually work in Node (when I tried using it, response.data was a string, not a Blob, since Node doesn't have Blobs), so this clarifies that this option should only be used in the browser
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.