Page MenuHomePhabricator

ApiQueryContentTranslation not respecting the limit parameter
Closed, ResolvedPublic1 Estimated Story Points

Description

ApiQueryContentTranslation is a ApiQueryGeneratorBase type class. But it does not take limit parameter as of now. It outputs all translations in one go. If the user has lot of translations, this will result a long list rendered in one go, with all api hits for images starting. This is not ideal since 100 such translations will initiate a 100 api hits for image/descriptions. We need to paginate it for example, load 10 translation and then while page scrolls load next set etc.

The dashboard loading should be fast as much as possible

Related Objects

Event Timeline

santhosh raised the priority of this task from to High.
santhosh updated the task description. (Show Details)
santhosh added a subscriber: santhosh.
Arrbee lowered the priority of this task from High to Medium.Aug 26 2015, 12:21 PM
Arrbee added a subscriber: Arrbee.
Arrbee raised the priority of this task from Medium to High.
Nikerabbit lowered the priority of this task from High to Medium.Sep 3 2015, 12:52 PM
Nikerabbit moved this task from Backlog to In Review on the LE-CX6-Sprint 3 board.
Nikerabbit edited a custom field.

Change 235720 had a related patch set uploaded (by Nikerabbit):
Implement limit parameter in ApiQueryContentTranslation

https://gerrit.wikimedia.org/r/235720

Change 235720 merged by jenkins-bot:
Implement limit parameter in ApiQueryContentTranslation

https://gerrit.wikimedia.org/r/235720

https://gerrit.wikimedia.org/r/235720 merged, that means, we show 100 items in the page now. Once we are done with refactorings of https://gerrit.wikimedia.org/r/#/c/232019/ we can proceed with 'infinite scroll' to load the remaining translations.

Change 239322 had a related patch set uploaded (by Santhosh):
API: Support limit and offset params for suggestion and translation queries

https://gerrit.wikimedia.org/r/239322

Change 239322 merged by jenkins-bot:
API: Support limit and offset params for suggestion and translation queries

https://gerrit.wikimedia.org/r/239322

Change 240068 had a related patch set uploaded (by Nikerabbit):
Dashboard: Show empty list information when suggestions or translations are empty

https://gerrit.wikimedia.org/r/240068

Change 240068 merged by jenkins-bot:
Dashboard: Show empty list information when suggestions or translations are empty

https://gerrit.wikimedia.org/r/240068

Verified that the list is loaded in sets as the user scrolls and reaches the end of the suggestions