Skip to content

Candidature: Ajout d’un nouveau motif de refus “Refus automatique” [GEN-2298]#5726

Merged
vincentporte merged 1 commit into
masterfrom
vp/GEN-2298-refus-auto-2sur3
Mar 5, 2025
Merged

Candidature: Ajout d’un nouveau motif de refus “Refus automatique” [GEN-2298]#5726
vincentporte merged 1 commit into
masterfrom
vp/GEN-2298-refus-auto-2sur3

Conversation

@vincentporte
Copy link
Copy Markdown

@vincentporte vincentporte commented Mar 5, 2025

🤔 Pourquoi ?

notion

🍰 ajout d'un motif de refus pour les candidatures, non accessible dans le formulaire de refus des employeurs.

🍰 Comment ?

🍦 Ajout dans l'enum RefusalReason et dans la liste des raisons de la classe hidden
🍦 pas de blocage dans l'admin

🚨 À vérifier

☑️ Mettre à jour le CHANGELOG_breaking_changes.md ?
☑️ Ajouter l'étiquette « Bug » ?

💻 Captures d'écran

image

@notion-workspace
Copy link
Copy Markdown

@vincentporte vincentporte self-assigned this Mar 5, 2025
@vincentporte vincentporte added the modifié Modifié dans le changelog. label Mar 5, 2025
@vincentporte vincentporte force-pushed the vp/GEN-2298-refus-auto-2sur3 branch 2 times, most recently from 84ad53d to 353049c Compare March 5, 2025 14:56
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Cf autres commentaires (je n'arrive pas à supprimer ce commentaire chapeau 🎩 😩 )

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tu pourrais éviter cette migration en allant directement ajouter ("auto", "Refus automatique"), dans itou/job_applications/migrations/0001_initial.py.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

C'est un peu étrange de rajouter ce message dans cette PR car a priori aucun code ne peut l'utiliser.
Il aurait a priori plus de sens dans la prochaine PR.

@vincentporte vincentporte force-pushed the vp/GEN-2298-refus-auto-2sur3 branch from 353049c to 313d634 Compare March 5, 2025 16:05
@vincentporte vincentporte added this pull request to the merge queue Mar 5, 2025
Merged via the queue into master with commit 5ede419 Mar 5, 2025
@vincentporte vincentporte deleted the vp/GEN-2298-refus-auto-2sur3 branch March 5, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants