-
Notifications
You must be signed in to change notification settings - Fork 103
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
Dm sort myplaces #2805
Dm sort myplaces #2805
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2805 +/- ##
========================================
Coverage 68.56% 68.56%
========================================
Files 86 86
Lines 7435 7435
========================================
Hits 5098 5098
Misses 2337 2337
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
A faire côté backend plutôt non ? |
Les deux se défendent. D'un côté, un order_by ne va pas nuire au performance de GeoNature, de l'autre, on peut réduire la charge côté serveur (même si c'est infime). |
Selon moi autant que l'API renvoie par défaut les objets par ordre alphabétique |
Ok, je ne savais pas ce que vous préfériez / quelle était la bonne pratique. Je modifie :) |
remove sorting in frontend
Sort places Alphabetically in API
Merci pour vos retours, c'est mis à jour faire le order_by coté backend |
Nickel ! Il reste juste à formater le code modifié avec Black. Si vous ne voyez pas comment faire, la documentation de Geonature indique la procédure à suivre https://docs.geonature.fr/development.html#backend. |
Bonjour @DonovanMaillard, j'ai fait ce qu'il fallait :) J'ai intégré ta modification dans le hotfix de la 2.13 #2791 |
Sans doute à reporter directement dans une hotfix? C'est simplement un "sort" sur la liste de "Mes lieux" pour faciliter la sélection.