Skip to content

Notifications : Correction de l'envoi des emails de nouvelle candidature reçue [GEN-2267]#5693

Merged
tonial merged 3 commits into
masterfrom
alaurent/fix_notifications
Feb 27, 2025
Merged

Notifications : Correction de l'envoi des emails de nouvelle candidature reçue [GEN-2267]#5693
tonial merged 3 commits into
masterfrom
alaurent/fix_notifications

Conversation

@tonial

@tonial tonial commented Feb 25, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

  • On ne veut pas les envoyer à des utilisateurs inactifs, même si le membership est encore actif.
  • On désactive à présent les membership en même temps que les utilisateurs dans l'admin
  • On corrige rétroactivement ls membership encore actifs alors que l'utilisateur ne l'est plus.

🍰 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 ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the bug label Feb 25, 2025
@tonial tonial self-assigned this Feb 25, 2025
@notion-workspace

Copy link
Copy Markdown

@tonial tonial added the modifié Modifié dans le changelog. label Feb 25, 2025
@tonial tonial force-pushed the alaurent/fix_notifications branch 3 times, most recently from ae8d347 to e4a3219 Compare February 27, 2025 05:14

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Merci beaucoup, ça règle tout 🙏

C’est voulu de ne pas avoir fait les membership des institutions ? Il n’y a pas de cas à corriger ?

Comment thread itou/prescribers/migrations/0009_deactivate_inactive_users_memberships.py Outdated
Comment thread itou/users/admin.py Outdated
Comment thread tests/www/apply/test_submit.py Outdated
Comment thread tests/www/apply/test_submit.py Outdated
Comment thread tests/www/apply/test_submit.py Outdated
Comment thread tests/users/test_admin.py Outdated
Comment thread tests/users/test_admin.py Outdated
Comment thread tests/users/test_admin.py Outdated
@tonial

tonial commented Feb 27, 2025

Copy link
Copy Markdown
Contributor Author

C'est dans le message du 3e commit: pas l'institutionnel dans ce cas, donc pas de migration pour eux

@tonial tonial force-pushed the alaurent/fix_notifications branch from e4a3219 to 8494c35 Compare February 27, 2025 09:45

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👨‍🍳 Merci encore !

@tonial tonial enabled auto-merge February 27, 2025 19:32
@tonial tonial force-pushed the alaurent/fix_notifications branch from 8494c35 to 6aa7e80 Compare February 27, 2025 19:36
@tonial tonial added this pull request to the merge queue Feb 27, 2025
Merged via the queue into master with commit 0750ce1 Feb 27, 2025
@tonial tonial deleted the alaurent/fix_notifications branch February 27, 2025 19:53
@francoisfreitag francoisfreitag changed the title Notifications: Correction de l'envoi des emails de nouvelle candidature reçue [GEN-2267] Notifications : Correction de l'envoi des emails de nouvelle candidature reçue [GEN-2267] Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants