search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Supprime le champ inutile Comment.update_index_date #6659

Closed philippemilink closed 1 month ago

philippemilink commented 1 month ago

Suppression du champ inutile Comment.update_index_date, devenu inutile depuis le passage à Typesense (il l'était peut-être même avant).

Rechercher toutes les occurrences de update_index_date, m'a fait découvrir qu'on déclenche une sauvegarde du Topic lorsqu'on édite un Post (ce qui crée un CommentEdit) pour marquer le Topic comme à réindexer, ce qui n'est pas (plus) nécessaire (cf mon commentaire dans le message de commit). J'en ai profité pour ajouter un petit test.

Contrôle qualité

Juste regarder le code et la CI.

coveralls commented 1 month ago

Coverage Status

coverage: 89.181% (-0.002%) from 89.183% when pulling 3eb842f33d68147c8b44ccd7313fa3316b3a2aa7 on philippemilink:remove_update_index_date into 56719a50592b2cda0cf34272025e96edef16adae on zestedesavoir:dev.