Candidats : Utiliser les données du Pilotage pour identifier les candidats dans la file active depuis plus de 30j#5813
Merged
Conversation
francoisfreitag
approved these changes
Mar 25, 2025
cc0006a to
68a9fc7
Compare
Contributor
Author
|
Retours pris en compte et j'ai ajouté un test, ce qui a permis de choper une erreur de variable dans les logs :). |
2bee44f to
8d9d7d7
Compare
8d9d7d7 to
cf56543
Compare
francoisfreitag
approved these changes
Apr 1, 2025
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤔 Pourquoi ?
Sinon mes "gentils" collègues vont fouetter des chats 😿 : #5790
🍰 Comment
Au final j'ai mis le champ sur
JobSeekerProfile(), le mettre surUser()me semblais aller à rebours de ce qu'on a fait dernièrement en rapatriant les champs uniquement destinés aux candidats depuis celui-ci. et on peux toujours faire unjobseeker_profile__is_stalled=Truesi besoin.Je me suis aussi posé la question de faire un champ plus générique (
ARRAY[TEXT]) afin de pouvoir ajouter d'autres marqueurs de ce style mais j'ai pas l'impression qu'on va en avoir besoin dans l'immédiat donc j'ai fait au plus simple.🏝️ Comment tester ?
Configurer
METABASE_API_KEYet lancer la commandeLe premier lancement après l'application de la migration prend un certain temps car PG doit réécrire les ~110k lignes, une fois cela fait la commande met 10s chez moi pour mettre à jour depuis zéro.