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

Fixed the collaborations response properties #149

Closed
wants to merge 1 commit into from

Conversation

mitquinn
Copy link
Contributor

@mitquinn mitquinn commented Apr 2, 2021

Removed pagination schema
Added next_marker property
Added previous_marker property

IMPORTANT: I would have used the marker_pagination like in the Folder Locks resource but there does not seem to have a limit property in this response.

Description

Please include a summary of the change and which issue is fixed.

If you are a Boxer, please also reference the related DDOC or APIWG tickets.
If you do not have a related Jira ticket and this is more than a bug fix, then
create one now.

Fixes #148

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own changes
  • I have run yarn lint to make sure my changes pass all linters
  • I have pulled the latest changes from the upstream developer branch

Contribution guidelines

For contribution guidelines, styleguide, and other helpful information please
see the CONTRIBUTING.md file in the root of this project.

Removed pagination schema
Added next_marker property
Added previous_marker property

IMPORTANT: I would have used the marker_pagination like in the Folder Locks resource but there does not seem to be a limit property in this response.
@mitquinn
Copy link
Contributor Author

mitquinn commented Apr 2, 2021

After more testing something is more strange here. I think it would be a mistake to change this resource.

It seems the List File Collaborations request has the wrong response not that this resource is in correct.

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.

Collaborations - Response contains wrong properties?
1 participant