Repository navigation
Fiche de poste : Le badge 20+ candidatures sur les résultats de recherche n’est pas compris et induit en erreur [GEN-2036] - #6422
Conversation
2a3a7fd to
f99c43c
Compare
dca213e to
11c11bd
Compare
tonial
left a comment
tonial
left a comment
There was a problem hiding this comment.
Je pense que tu peux squasher un grand nombre de commits (voir tous les commits à l'exception peut être de job_descriptions: limit badge logic to recent jobapps)
Il manque quelques tests sur ce commit là
|
|
||
| MAX_UI_RANK = 32767 | ||
| OVERWHELMED_THRESHOLD = 20 | ||
| UNPOPULAR_THRESHOLD = 0 |
There was a problem hiding this comment.
C'est dommage de ne pas mettre tout de suite 2 ici (peut être que la valeur a changé au cours du développement du ticket, en tout cas, ça vaut le coup de mettre la bonne valeur dès le début en fusionnant les commits suivants qui changent sa valeur)
There was a problem hiding this comment.
Ok je vais squasher.
8e15161 to
54c9817
Compare
54c9817 to
7873149
Compare
tonial
left a comment
tonial
left a comment
There was a problem hiding this comment.
👍
Je te laisse ignorer mon commentaire si tu n'es pas d'accord :)
| def with_annotation_is_unpopular(self): | ||
| annotation = self.with_job_applications_count().annotate( | ||
| # Avoid a circular import | ||
| job_application_model = self.model._meta.get_field("jobapplication").related_model |
There was a problem hiding this comment.
Je pense que je trouve from itou.job_applications.models import JobApplication plus simple par rapport à aller chercher un accès via les _meta du modèle
There was a problem hiding this comment.
En fait je suis resté iso avec ce qui est fait dans with_count_recent_received_job_apps dans le même fichier. Je vais regarder si on peut tout simplifier.
There was a problem hiding this comment.
Nope :
ImportError: cannot import name 'CompanyMembership' from partially initialized module 'itou.companies.models' (most likely due to a circular import)
Bref je MEP tel quel.
There was a problem hiding this comment.
Je proposais de remettre ce qu'il y avait avant ta PR :
Tu importes le modèle directement ici
2 participants
🤔 Pourquoi ?
Au lieu d’indiquer
20+ candidatures, afficher le badgeSoyez parmi les premiers à postulersur les postes qui ont reçu N ou moins candidatures récentes.N=0 pour commencer. Finalement N=1 et ce sera facile à ajuster (cf troisième commit).
🍰 Comment ?
On change la logique du badge existant : au lieu de le montrer si 20 ou plus candidatures, on le montre si N ou moins candidatures.
Dans le futur on pourra facilement ajuster le seuil N.
💻 Captures d'écran
recherche employeurs
recherche fiches de poste