Proposed API method for recursive directory loading #28

Closed
Marak opened this Issue Jun 8, 2012 · 2 comments

Comments

Projects
None yet
2 participants
@Marak
Contributor

Marak commented Jun 8, 2012

Hail -

I've been experimenting with your node-dbox library and have been enjoying using it so far.

In my application I've created a recursive readdir method that can crawl all files and directories in a dropbox folder path using node-dbox. It has a some-what configurable concurrency delay ( as to not spam dropbox api ). It also can create an object hash of file paths with getter methods for file reading, so that you may crawl first, and load files lazily later.

Would you be willing to accept such a feature to node-dbox? If so, I can try to do a pull request.

First day using dropbox API, so I'm hoping I didn't miss something obvious...

@sintaxi

This comment has been minimized.

Show comment Hide comment
@sintaxi

sintaxi Jun 8, 2012

Owner

Definitely open to that. I think something like that would be very useful.

Owner

sintaxi commented Jun 8, 2012

Definitely open to that. I think something like that would be very useful.

@Marak

This comment has been minimized.

Show comment Hide comment
@Marak

Marak Jun 8, 2012

Contributor

Okay! I'll see what I can do.

On Fri, Jun 8, 2012 at 11:19 AM, Brock Whitten <
reply@reply.github.com

wrote:

Definitely open to that. I think something like that would be very useful.


Reply to this email directly or view it on GitHub:
sintaxi#28 (comment)

Marak Squires
Co-founder and Chief Evangelist
Nodejitsu, Inc.
marak.squires@gmail.com

Contributor

Marak commented Jun 8, 2012

Okay! I'll see what I can do.

On Fri, Jun 8, 2012 at 11:19 AM, Brock Whitten <
reply@reply.github.com

wrote:

Definitely open to that. I think something like that would be very useful.


Reply to this email directly or view it on GitHub:
sintaxi#28 (comment)

Marak Squires
Co-founder and Chief Evangelist
Nodejitsu, Inc.
marak.squires@gmail.com

Marak added a commit to Marak/node-dbox that referenced this issue Jun 10, 2012

@ghost ghost assigned sintaxi Jul 9, 2012

@sintaxi sintaxi closed this Jul 26, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment