box_fetch() seems to be limited maximum 100 files per directory about boxr HOT 7 CLOSED

Did some first digging -

https://docs.box.com/reference#get-folder-info

seems that the call used in box_ls() is limited to 100 by default, but can be extended to 1000 - otherwise needs pagination.

If it's OK, I'll do a provisional PR to set default to 1000, but will ultimately need to use pagination.

Thoughts?

from boxr.

Thanks for spotting this --- a PR would be much appreciated!

from boxr.

Will do!

from boxr.

👍 Thanks so much, you've been added as the package's first contributor! (b113ae5).

Agree that pagination is the way to solve this properly, thanks for offering to contribute it. I wrote a very simple pagination function for box_search here. If you think you can adapt/extend/genericize that, by all means do.

I think the improvements probably deserve a test. If you'd like to contribute one, shoot me a mail at [email protected], and I'll send you the creds for the boxr account I use for testing (don't use your personal box account; it'll overwrite and delete everything several times!). However, if you can't spare the time to write a test of two, don't let that put you off.

from boxr.

• Glad to contribute, as I've said before, this package is a lifesaver!
• Will be happy to take a stab at pagination, following your lead.
• I'll send a separate email to you to start the ball rolling on testing.

from boxr.

fixed in 866f03b

from boxr.

🎉

from boxr.