Problème dans la suppression des hooks

[mathieutu]

@mathieutu

Je pense qu'il faut passer les soft-deleted dans deleted avec un tag quelconque. Ça semble plus "normal" que de checker dans une modif de l'user.

@D34THWINGS

@mathieutu le problème du soft delete c'est que c'est une action d'edit. C'est même pas vraiment un soft delete, c'est de la désactivation planifiée. Il faudrait limite créer un job qui se déclanche le jour correspondant au leftAt du user, ce qui est tout de suite bien plus compliqué.

@mathieutu

Ah, du coup c'est un leaveAt en fait pas un leftAt ? Tu le remplis avant, et tu prends juste ceux qui sont après le jour de la requête ?
Parce que du coup ça marche pas avec les hooks, parce que sinon c'est au service client du hook en question de devoir faire son cron pour virer l'user à la date spécifiée.
Parce que Morgan est d'accord sur le coup, tu supprimes l'user, nous osef de savoir si c'est du soft ou pas.

@D34THWINGS

Pour le coup c'est pas une suppression, c'est juste que la personne est partie de l'entreprise. On peut pas supprimer les compte LVConnect parce que ça fou trop le bordel de manière générale. Ou alors il faut s'assurer que chaque app qui utilise LVConnect possède une copie du user en local. Le leftAt indique juste que la personne ne fait plus partie de l'entreprise à tel date. Les apps après en font ce qu'elles veulent, en l'occurence LVConnect empêche les tokens d'être générés après cette date.
D'ailleurs je viens de penser au fait qu'il faut que je révoque tous les tokens en cours passé cette date 😅 .

[mathieutu]

@D34THWINGS, ce que je disais c'est que ce n'est pas du tout du soft delete.

De ce que je comprends la date n'est pas set le jour du départ, via une action manuelle, du coup ce n'est pas un vrai leftAt, qui dit que simplement s'il y une date dans le champs alors l'user est supprimé.

Là c'est un leaveAt. Un champs que tu programmes à l'avance et que tu checkes à chaque requête en fonction de la date de la requête. Si c'est le mode de fonctionnement choisis pour LvConnect pas de pb, mais on ne doit pas l'imposer aux autres applis.

Parce que là chaque appli qui veut faire un vrai soft delete, avec juste un flag "deleted(_at)", ou même supprimer complètement l'user (il a pas forcément besoin de le stocker de son côté lui), elle est obligée de stocker l'information du leaveAt quelque part, de faire un cron tous les jours de son côté, et de supprimer l'user dans ce cron.

Du coup il vaut mieux faire cette opération une seule fois via le hook dans lvconnect que dans toutes les applis.

[D34THWINGS]

@mathieutu la problématique c'était de s'assurer que tous les comptes soient désactivés à la date de sortie de contrat. Du coup, pour s'assurer que ça soit bien le cas j'ai gérer ça côté LVConnect. Mais on peut très bien déporter cette logique côté applicatif et laisser les apps gérer la désactivation du compte de leur côté. Pour le coup je ne sais pas quelle est la meilleure méthode à adopter.