Allow client to choose pagination #40
Conversation
This is the same before, except that client can override number of results per page. A maximum limit is defined, though
|
LGTM niveau code (pas encore testé), un point de détail though : il manque un test pour vérifier qu'il n'y a pas de régression. Au pire si tu as besoin de merge on peut vu qu'on n'est pas encore exactement parfaits niveau tests, mais ça peut être cool de commencer à se forcer à ajouter des tests relevant à chaque PR :) |
|
Ah, un autre point de détail : là les commentaires me semblent superflus : les variables étant bien nommées, le code semblerait aussi clair et plus rapide à lire sans. (same as before, on peut merger et faire un commit style fix sur develop, ou argumenter pendant de longues heures sur l'intérêt de ces commentaires ^^) |
|
Pas besoin de merger cette PR tout de suite. On peut écrire les tests d'abord. |
|
Pour info, cette syntaxte est deprecated. |
This is the same before, except that client can override number of
results per page.
A maximum limit is defined, though