Page MenuHomeSoftware Heritage

/api/1/revision/log/ - Add pagination
Closed, MigratedEdits Locked

Description

To this day, there is only a limit (propery configuration) to what's returned.
Remove such limit.
Add pagination following the pagination installed in T605 on /api/1/content/symbol/<expression>/.

Event Timeline

ardumont mentioned this in Unknown Object (Maniphest Task).
zack moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Jan 12 2017, 12:22 PM
zack added a subscriber: zack.

To this day, there is only a limit (propery configuration) to what's returned.
Remove such limit.

I don't know the details of that limit, but we might want to keep it too, as safeguard, in addition to pagination. To be discussed…

Add pagination following the pagination installed in T605 on /api/1/content/symbol/<expression>/.

Yes! Ideally, we should have a single implementation of the pagination code, which is easy/trivial to plug into any endpoint that needs it.

zack raised the priority of this task from Normal to High.Jan 13 2017, 10:13 AM

As explained on irc, there is actually a pagination.
The last element is used to create a new /revision/log/ link to follow in the 'next_urls_rev' key.

I tried and improved the docstring instead.