Closed rap2hpoutre closed 6 years ago
@vinyll Merci beaucoup pour ta review. As-tu pu lancer la migration en local pour vérifier que ça ne pétait pas ? (normalement c'est bon, mais je suis toujours méfiant car débutant en mongo)
@vinyll Je te laisserai ensuite mettre la PR en validé avant que je merge...
J'ai testé la migration, ca semble apriori correct.
@vinyll OK alors je lance la migration et je merge après le meeting
L'objectif de cette PR est de permettre de rechercher des actions de la même manière qu'on recherche des acteurs actuellement.
À noter :
location
a été renommée enloc
pour être cohérent avec le fonctionnement des acteurs. Lors de l'ajout d'un acteur, on définit aussi leloc
de l'action automatiquement (on a oublié de le faire, car notre modèle de données parlait d'unelocation
pour les actions, et potentiellemnt plusieurs plus tard), ce qui permet de faire des recherches paractions
indépendamment des acteurs.loc
pour toutes les actions. @vinyll Pourras-tu tester cette migration ? Il faudra qu'on la lance en prod avant de merger.Attention, cette PR nécessite pas mal d'attention car elle modifie 15 fichiers (mais beaucoup de tests unitaires, donc ça limite)
@vinyll Dis moi quand tu es dispo pour relire cette PR, au plaisir de lire ta review !