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

feat(api): 121 - add coherence check for missing IN_PROGRESS status fo… #4693

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

@Johannbr Johannbr force-pushed the feat-121-in_progress-missing branch from 4960e75 to c35f785 Compare January 23, 2025 17:12
@Johannbr Johannbr changed the title feat(api) 121 - add coherence check for missing IN_PROGRESS status fo… feat(api): 121 - add coherence check for missing IN_PROGRESS status fo… Jan 23, 2025
@Johannbr Johannbr force-pushed the feat-121-in_progress-missing branch from c35f785 to eeb4d96 Compare January 23, 2025 17:13
Copy link
Collaborator

@C2Chandelier C2Chandelier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rien a dire sur le CRONS.
Par contre tu n'as pas identifié de trou dans la raquette au niveau de la validation des jeune directement en LP sans passer par In_PROGRESS ?
(il me semble qu'on avait déjà réglé cette histoire mais bon...)

@Johannbr
Copy link
Collaborator Author

Rien a dire sur le CRONS. Par contre tu n'as pas identifié de trou dans la raquette au niveau de la validation des jeune directement en LP sans passer par In_PROGRESS ? (il me semble qu'on avait déjà réglé cette histoire mais bon...)

Oui, il y a eu un fix apporté en septembre mais à priori il y a plus d'un trou dans la raquette. Ce cron a pour objectif de permettre de surveiller plus régulièrement ce problème.

@Johannbr Johannbr merged commit be7664b into main Jan 31, 2025
17 checks passed
@Johannbr Johannbr deleted the feat-121-in_progress-missing branch January 31, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants