Skip to content

GEIQ: ajout des DDETS/DREETS en copie de l'email de notification de contrôle définitif [GEN-2522] - #6285

Merged
xavfernandez merged 1 commit into
masterfrom
xf/geiq_assessment_final_review_with_cc
Jun 10, 2025
Merged

GEIQ: ajout des DDETS/DREETS en copie de l'email de notification de contrôle définitif [GEN-2522]#6285
xavfernandez merged 1 commit into
masterfrom
xf/geiq_assessment_final_review_with_cc

Conversation

@xavfernandez

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Car elles veullent être en copie de l'email.

🍰 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

@xavfernandez xavfernandez self-assigned this Jun 6, 2025
@xavfernandez xavfernandez added the modifié Modifié dans le changelog. label Jun 6, 2025
@notion-workspace

Copy link
Copy Markdown

)
.select_related("user")
}
email_message.cc = sorted(cc_user.email for cc_user in cc_users)

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.

On a toujours une limite de Mailjet à 50 TO, CC, BCC. Pour le moment, l’institution avec le plus de membres en a moins que ça, mais tu peux faire comme moi et ajouter à ta TODO de gérer ce problème. Peut-être qu’en passant à Brevo, le problème disparaîtra.

SELECT institution_id, count(user_id)
FROM institutions_institutionmembership
GROUP BY institution_id
ORDER BY 2 DESC;

@xavfernandez
xavfernandez force-pushed the xf/geiq_assessment_final_review_with_cc branch from fd5c68b to 4280cd2 Compare June 10, 2025 12:55
@xavfernandez
xavfernandez enabled auto-merge June 10, 2025 12:56
@xavfernandez
xavfernandez added this pull request to the merge queue Jun 10, 2025
Merged via the queue into master with commit 36cfd2a Jun 10, 2025
@xavfernandez
xavfernandez deleted the xf/geiq_assessment_final_review_with_cc branch June 10, 2025 13:12
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.

3 participants