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

Candidature: ajout d'une notification au candidat et au prescripteur/orienteur éventuel lors d'une mise en attente [GEN-1759] #4387

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

xavfernandez
Copy link
Contributor

@xavfernandez xavfernandez commented Jul 8, 2024

🤔 Pourquoi ?

Pour tenter de réduire le nombre de candidature restant bloquée pour cause de candidat injoignable ou ne se présentant pas à l'entretien.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

https://c1-review-xfernandez-postponed-notification.cleverapps.io/

Mettre des candidatures en attente et aller voir dans l'admin les mails générés.

💻 Captures d'écran

@xavfernandez xavfernandez added the ajouté Ajouté dans le changelog. label Jul 8, 2024
@xavfernandez xavfernandez requested review from leo-naeka and tonial July 8, 2024 12:33
@xavfernandez xavfernandez self-assigned this Jul 8, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jul 8, 2024
@xavfernandez xavfernandez force-pushed the xfernandez/postponed_notification branch from a396145 to a56a6fe Compare July 8, 2024 12:50
Copy link

github-actions bot commented Jul 8, 2024

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Copy link
Contributor

@leo-naeka leo-naeka left a comment

Choose a reason for hiding this comment

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

👌

Ce n'est pas spécifié sur la carte : les notifications sont bien désactivables par les utilisateurs ou leur envoi est forcé ? Dans le second cas il faut passer can_be_disabled à False pour ces 2 notifications

itou/job_applications/notifications.py Show resolved Hide resolved
@xavfernandez
Copy link
Contributor Author

Ce n'est pas spécifié sur la carte : les notifications sont bien désactivables par les utilisateurs ou leur envoi est forcé ? Dans le second cas il faut passer can_be_disabled à False pour ces 2 notifications

Bon point, je m'étais dit qu'on souhaitait laisser la main aux utilisateurs mais je vais demander confirmation 👍

@xavfernandez xavfernandez force-pushed the xfernandez/postponed_notification branch from a56a6fe to e498d68 Compare July 10, 2024 13:07
@xavfernandez
Copy link
Contributor Author

Bien vu Léo, on ne souhaitait pas leur laisser le choix: https://itou-inclusion.slack.com/archives/CU8ATF54L/p1720616756435209?thread_ts=1720443268.159019&cid=CU8ATF54L
J'ai rajouté les can_be_disabled.

@xavfernandez xavfernandez added this pull request to the merge queue Jul 10, 2024
Merged via the queue into master with commit 1a83f49 Jul 10, 2024
11 checks passed
@xavfernandez xavfernandez deleted the xfernandez/postponed_notification branch July 10, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants