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

NEW MAJOR VERSION - added large file support and upload progress. #22

Merged
merged 11 commits into from
Mar 15, 2017
Merged

NEW MAJOR VERSION - added large file support and upload progress. #22

merged 11 commits into from
Mar 15, 2017

Conversation

IvanKalinin
Copy link
Contributor

switched form node-request to promise based library 'axios';
added methods for large files;
added on download and on upload options;
updated tests

added methods for large files;
added on download and on upload options;
updated tests
@crazyscience
Copy link
Contributor

@yakovkhalinsky Hold on for a bit before you merge this one in.

@crazyscience
Copy link
Contributor

@yakovkhalinsky Need to update the Readme and probably bring you up to speed with the changes.

@yakovkhalinsky
Copy link
Owner

Excellent! Very much appreciate this massive contribution.

@crazyscience
Copy link
Contributor

Closed for further testing and verification. New PR will be available after extensive testing, both unit and human.

@IvanKalinin IvanKalinin reopened this Mar 14, 2017
@crazyscience
Copy link
Contributor

Needs to include q in package.json.

included `q` in dependencies
@crazyscience
Copy link
Contributor

@yakovkhalinsky ok. ready to go.

@yakovkhalinsky
Copy link
Owner

Very nice work mate 👍

Merging

@yakovkhalinsky
Copy link
Owner

Will npm publish when I get home in ~5 hours :)

@yakovkhalinsky yakovkhalinsky merged commit 885d9c2 into yakovkhalinsky:master Mar 15, 2017
@yakovkhalinsky yakovkhalinsky modified the milestone: 1.0.0 release Mar 15, 2017
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.

3 participants