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

download method: optional path to write file #22

Open
chaps opened this issue Apr 13, 2018 · 2 comments
Open

download method: optional path to write file #22

chaps opened this issue Apr 13, 2018 · 2 comments

Comments

@chaps
Copy link
Contributor

chaps commented Apr 13, 2018

When calling the download method ,
a file object is created with the same path as the requested file to download ,
paths like /Documents/somefile won't exist usually under the local filesystems where the
library is being used .

The default path to write the file contents could be the local CWD plus the name of the file being downloaded , and the method could accept a keyword argument in order to explicitly tell the desired path where to write the file contents.

@cabarnes
Copy link
Member

@chaps Is this what you're looking for? #17

@chaps
Copy link
Contributor Author

chaps commented Apr 13, 2018

Hi @cabarnes ,
I'm looking for something more like this : chaps@89a816a
let me know if this fits in order to submit a PR
thanks!

@chaps chaps changed the title download method local file path download method: optional path to write file Apr 13, 2018
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

No branches or pull requests

2 participants