APIv2 : Add filter by ratings on outdoor courses and sites #2768
Conversation
|
Test summary
Run details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
Codecov Report
@@ Coverage Diff @@
## master #2768 +/- ##
=======================================
Coverage 97.27% 97.27%
=======================================
Files 287 287
Lines 19884 19916 +32
=======================================
+ Hits 19342 19374 +32
Misses 542 542
Continue to review full report at Codecov.
|
| # Exclude if ratings in hierarchy don't match any wanted ratings | ||
| found_ratings = site.super_ratings_id | ||
| if found_ratings.isdisjoint(wanted_ratings): | ||
| queryset = queryset.exclude(id=site.pk) |
There was a problem hiding this comment.
Je sais que ce n'est pas idéal d'itérer sur le queryset, mais vu la complexité pour récupérer les pratiques et les cotations des sites enfants, je n'ai pas d'autre idées
There was a problem hiding this comment.
Ok, pour le moment on valide comme ça. Si ca pose des problèmes de perf on prendra le temps de réfléchir à une meilleure manière de faire alors.
Met juste un commentaire en TODO dans le code qu'on puisse savoir que c'est optimisable.
| # Exclude if ratings in hierarchy don't match any wanted ratings | ||
| found_ratings = site.super_ratings_id | ||
| if found_ratings.isdisjoint(wanted_ratings): | ||
| queryset = queryset.exclude(id=site.pk) |
There was a problem hiding this comment.
Ok, pour le moment on valide comme ça. Si ca pose des problèmes de perf on prendra le temps de réfléchir à une meilleure manière de faire alors.
Met juste un commentaire en TODO dans le code qu'on puisse savoir que c'est optimisable.
No description provided.