Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

correctif(ResetExpiringDossiersJob): timeout sur la prod, utilise une requete plus générique mais plus rapide [on ne scope pas aux dossiers ayant ete notifies, on les reset tous car ca timeouté #9797

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Dec 5, 2023

No description provided.

@colinux
Copy link
Member

colinux commented Dec 5, 2023

Je pige pas: là tu reset systématiquement tous les dossiers de la démarche, non ?

Par contre OK pour récupérer tous les dossiers, mais filtrer en ruby ceux pour lesquels on veut faire le reset. (juste que ça fera probablement 1 req/dossier plutôt qu'un update all, mais c'est moins génant)

@mfo mfo force-pushed the US/fix-reset-expiring-dossiers-job branch from 01103ff to 5902a19 Compare December 6, 2023 15:38
@mfo
Copy link
Contributor Author

mfo commented Dec 6, 2023

@colinux , tx! MAJ, je trouvais ça préférable de tous les mettre a jour quite a faire une noop. Mais bon pr moi si ça passe ainsi go (et j'ai fais un update all via un filtre, histoire de pas trop solliciter la bdd)

… requete plus générique mais plus rapide [on ne scope pas aux dossiers ayant ete notifies, on les reset tous car ca timeouté
@mfo mfo force-pushed the US/fix-reset-expiring-dossiers-job branch from 5902a19 to b0e818f Compare December 6, 2023 15:45
@colinux
Copy link
Member

colinux commented Dec 7, 2023

@mfo by bad, je crois que j'avais mal compris le besoin. Là ce que tu dis c'est qu'on est vraiment dans un cas où veut reset toutes les expirations de tous les dossiers de la démarche, et donc renvoyer les emails de notice etc… (c'est cet aspect là que j'avais mal pigé, où je craignais de doubler des emails, mais la feature c'est justement de tout reset de nouveau).
En tout cas ça me va :)

@mfo mfo added this pull request to the merge queue Dec 7, 2023
@mfo
Copy link
Contributor Author

mfo commented Dec 7, 2023

avant le fait de changer la durée de conservation n'arretait pas l'expiration des dossier. ce job est la pour remettre à 0 ces flag sinon ca va finir expiré. tx pr tes feedback

Merged via the queue into main with commit 6a4bb06 Dec 7, 2023
15 checks passed
@mfo mfo deleted the US/fix-reset-expiring-dossiers-job branch December 7, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants