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

[NOTIFICATION] informer les utilisateurs des nouvelles réponses à leur question #331

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Jun 21, 2023

Description

🎸 Email quotidien aux utilisateurs ayant posé une question dans la communauté pour les informer du nombre de nouvelles réponses depuis le dernier email.

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).

Points d'attention

🦺 Requete dans collect_following_replies pour compter les posts créer après la derniere notification et qui ne sont pas le premier post (= question d'origine)
🦺 La derniere notification est enregistrée dans le modèle EmailSentTrack, avec le kind following_replies
🦺 Attention, un object email_sent_track est créé pour chaque notification de chaque Topic, mais la tâche recherche la dernière notification, tous topic confondus. Postulat : il n'y a pas de réponses plus anciennes que le plus récent email_sent_track qui n'aurait pas fait l'objet d'une notification elle-même.

Captures d'écran (optionnel)

image

@vincentporte vincentporte added the python Pull requests that update Python code label Jun 21, 2023
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Le survol de révèle pas de feu. ✈️

lacommunaute/notification/tests/tests_tasks.py Outdated Show resolved Hide resolved
lacommunaute/notification/tests/tests_tasks.py Outdated Show resolved Hide resolved
lacommunaute/notification/utils.py Outdated Show resolved Hide resolved
@vincentporte vincentporte force-pushed the 330-notification-informer-les-utilisateurs-des-nouvelles-réponses-à-leur-question branch from b69d9a3 to ebddab5 Compare June 22, 2023 11:36
@vincentporte vincentporte merged commit b81e0a1 into master Jun 22, 2023
4 checks passed
@vincentporte vincentporte deleted the 330-notification-informer-les-utilisateurs-des-nouvelles-réponses-à-leur-question branch June 22, 2023 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[NOTIFICATION] informer les utilisateurs des nouvelles réponses à leur question
2 participants