Skip to content
This repository was archived by the owner on Nov 23, 2024. It is now read-only.

idk is this a good idea?#175

Draft
gregziegan wants to merge 1 commit into
masterfrom
greg/paginated-apis
Draft

idk is this a good idea?#175
gregziegan wants to merge 1 commit into
masterfrom
greg/paginated-apis

Conversation

@gregziegan

@gregziegan gregziegan commented Apr 11, 2019

Copy link
Copy Markdown
Contributor

all collection APIs now return the following shape:

{
  collection_count: 123,
  collection: [
    ...
  ]
}

I thought about cursor-based but wasn't sure that it was necessary. Not sure that this is necessary.

Will fix tests if people like this will close if we have another idea.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant