Trier les fiches de détection par numéro séquentiel et non par ID #755
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
alanzirek
commented
Feb 25, 2025
•
alanzirek
commented
- Sélectionner la première fiche de détection par numéro et non par ID dans la vue détail événement
- MAJ des tests avec factories
- Remplacement de baker par des factories dans quelques tests
- Ajout d'une méthode create_miniaml pour créer une factory Lieu ou Prelevement avec uniquement les champs obligatoires
- Ajout de la factory PosistionChaineDistribution
alanzirek
force-pushed
the
detection-sort-by-number-not-id
branch
from
3c2b965 to
d74ebde
Compare
Anto59290
reviewed
Feb 27, 2025
sv/views.py
Outdated
| return ( | ||
| Evenement.objects.all() | ||
| .annotate(first_detection_id=Min("detections__id")) | ||
| .annotate(first_detection_id=Subquery(first_detections_subquery)) |
There was a problem hiding this comment.
Je ne sais pas dans quelle mesure ma remarque est liée uniquement à cette PR mais j'ai l'impression que first_detection_id ne sert que pour active_detection, pourquoi ne prend ton pas la première détection du queryset si celui ci est tri plutôt que d'annotate l'objet avec une subquery, est ce que ça ne serait pas plus performant ?
There was a problem hiding this comment.
oui 👍 J'ai remplacé par un Prefetch qui précharge les détections déjà triées correctement + sélection de la 1ère détection via self.object.detections.first() (cf commit 84d2ebd).
J'ai aussi ajouté un test l'affichage des détections triées par numéro et non par id.
Anto59290
approved these changes
Feb 28, 2025
alanzirek
force-pushed
the
detection-sort-by-number-not-id
branch
2 times, most recently
from
36b41f7 to
51dccba
Compare
- Sélectionner la première fiche de détection par numéro et non par ID dans la vue détail événement - MAJ des tests avec factories * Remplacement de baker par des factories dans quelques tests * Ajout d'une méthode create_miniaml pour créer une factory Lieu ou Prelevement avec uniquement les champs obligatoires * Ajout de la factory PosistionChaineDistribution
alanzirek
force-pushed
the
detection-sort-by-number-not-id
branch
from
51dccba to
12d6916
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.