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

Routes, offsets et pages #229

Closed
camillemonchicourt opened this issue Jul 1, 2020 · 2 comments
Closed

Routes, offsets et pages #229

camillemonchicourt opened this issue Jul 1, 2020 · 2 comments

Comments

@camillemonchicourt
Copy link
Member

Certaines routes ont un paramètre de LIMITE et de OFFSET. Et cela a été ajouté sur la route allnamebylist (#208) pour que l'application Sync-mobile puisse récupérer les taxons par lots.

Mais le paramètre offset fonctionne en fait comme un paramètre de page (

page = int(request.args.get("offset", 0))
). Or dans Sync-mobile le paramètre est utilisé comme un offset (si je limite à 1000, pour obtenir le contenu de la 2° page alors offset=1000).

Pour être plus correct, on propose donc d'utiliser le paramètre offset actuel comme un vrai offset et d'ajouter un paramètre page correspondant au fonctionnement actuel.

C'est le cas dans 3 ou 4 routes. A priori pas de conséquences dans TaxHub ni GeoNature qui n'utilise pas actuellement ce paramètre sur ces routes.

A faire aussi ensuite dans GeoNature où cela a plus de conséquences.

@camillemonchicourt
Copy link
Member Author

Fait dans 7d41cc9 et a368680

@camillemonchicourt
Copy link
Member Author

Réalisé dans la 1.7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant