Skip to content

Conversation

@trysmr
Copy link
Contributor

@trysmr trysmr commented May 13, 2023

Hi,

The List folder collaborations API does not seem to support offset and limit.
https://developer.box.com/reference/get-folders-id-collaborations/

If we specify an offset and limit smaller than total_count for the folder_collaborations, the loop in the get_all_with_pagination will increase the number of collaborators.

We should use the the get method in the folder_collaborations.

Thank you!

@xhocquet
Copy link
Collaborator

Hey @trysmr , thanks for the contribution! Your change makes sense based on the info provided. I'll take a look and see if we can get this merged in

@xhocquet xhocquet force-pushed the folder_collaborations branch from 94c62c1 to 3facabc Compare September 15, 2025 00:50
@xhocquet
Copy link
Collaborator

Went ahead and updated tests + finished the implementation. This endpoint supports marker pagination so I mirrored the file_collaborations implementation

@xhocquet xhocquet merged commit 21953cb into cburnette:master Sep 15, 2025
6 checks passed
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.

2 participants