diff --git a/bda/admin.py b/bda/admin.py index 7a981d1c..2aec82c9 100644 --- a/bda/admin.py +++ b/bda/admin.py @@ -1,10 +1,11 @@ from datetime import timedelta -from custommail.shortcuts import send_mass_custom_mail from dal.autocomplete import ModelSelect2 from django import forms from django.contrib import admin +from django.core.mail import send_mass_mail from django.db.models import Count, Q, Sum +from django.template import loader from django.template.defaultfilters import pluralize from django.utils import timezone @@ -169,19 +170,23 @@ class ParticipantAdmin(ReadOnlyMixin, admin.ModelAdmin): form = ParticipantAdminForm def send_attribs(self, request, queryset): - datatuple = [] + emails = [] for member in queryset.all(): + subject = "Résultats du tirage au sort" attribs = member.attributions.all() context = {"member": member.user} - shortname = "" + + template_name = "" if len(attribs) == 0: - shortname = "bda-attributions-decus" + template_name = "bda/mails/attributions-decus.txt" else: - shortname = "bda-attributions" + template_name = "bda/mails/attributions.txt" context["places"] = attribs - print(context) - datatuple.append((shortname, context, "bda@ens.fr", [member.user.email])) - send_mass_custom_mail(datatuple) + + message = loader.render_to_string(template_name, context) + emails.append((subject, message, "bda@ens.fr", [member.user.email])) + + send_mass_mail(emails) count = len(queryset.all()) if count == 1: message_bit = "1 membre a" diff --git a/bda/models.py b/bda/models.py index ffc203ed..80790746 100644 --- a/bda/models.py +++ b/bda/models.py @@ -2,14 +2,14 @@ import calendar import random from datetime import timedelta -from custommail.models import CustomMail -from custommail.shortcuts import send_mass_custom_mail from django.conf import settings from django.contrib.auth.models import User from django.contrib.sites.models import Site from django.core import mail +from django.core.mail import EmailMessage, send_mass_mail from django.db import models from django.db.models import Count, Exists +from django.template import loader from django.utils import formats, timezone @@ -116,16 +116,19 @@ class Spectacle(models.Model): bda_generic.nb_attr = 1 members.append(bda_generic) # On écrit un mail personnalisé à chaque participant - datatuple = [ + mails = [ ( - "bda-rappel", - {"member": member, "nb_attr": member.nb_attr, "show": self}, + str(self), + loader.render_to_string( + "bda/mails/rappel.txt", + context={"member": member, "nb_attr": member.nb_attr, "show": self}, + ), settings.MAIL_DATA["rappels"]["FROM"], [member.email], ) for member in members ] - send_mass_custom_mail(datatuple) + send_mass_mail(mails) # On enregistre le fait que l'envoi a bien eu lieu self.rappel_sent = timezone.now() self.save() @@ -348,21 +351,24 @@ class SpectacleRevente(models.Model): BdA-Revente à tous les intéressés. """ inscrits = self.attribution.spectacle.subscribed.select_related("user") - datatuple = [ + mails = [ ( - "bda-revente", - { - "member": participant.user, - "show": self.attribution.spectacle, - "revente": self, - "site": Site.objects.get_current(), - }, + "BdA-Revente : {}".format(self.attribution.spectacle.title), + loader.render_to_string( + "bda/mails/revente-new.txt", + context={ + "member": participant.user, + "show": self.attribution.spectacle, + "revente": self, + "site": Site.objects.get_current(), + }, + ), settings.MAIL_DATA["revente"]["FROM"], [participant.user.email], ) for participant in inscrits ] - send_mass_custom_mail(datatuple) + send_mass_mail(mails) self.notif_sent = True self.notif_time = timezone.now() self.save() @@ -373,20 +379,23 @@ class SpectacleRevente(models.Model): leur indiquer qu'il est désormais disponible au shotgun. """ inscrits = self.attribution.spectacle.subscribed.select_related("user") - datatuple = [ + mails = [ ( - "bda-shotgun", - { - "member": participant.user, - "show": self.attribution.spectacle, - "site": Site.objects.get_current(), - }, + "BdA-Revente : {}".format(self.attribution.spectacle.title), + loader.render_to_string( + "bda/mails/revente-shotgun.txt", + context={ + "member": participant.user, + "show": self.attribution.spectacle, + "site": Site.objects.get_current(), + }, + ), settings.MAIL_DATA["revente"]["FROM"], [participant.user.email], ) for participant in inscrits ] - send_mass_custom_mail(datatuple) + send_mass_mail(mails) self.notif_sent = True self.notif_time = timezone.now() # Flag inutile, sauf si l'horloge interne merde @@ -418,31 +427,30 @@ class SpectacleRevente(models.Model): "show": spectacle, } - c_mails_qs = CustomMail.objects.filter( - shortname__in=[ - "bda-revente-winner", - "bda-revente-loser", - "bda-revente-seller", - ] - ) - - c_mails = {cm.shortname: cm for cm in c_mails_qs} + subject = "BdA-Revente : {}".format(spectacle.title) mails.append( - c_mails["bda-revente-winner"].get_message( - context, + EmailMessage( + subject=subject, + body=loader.render_to_string( + "bda/mails/revente-tirage-winner.txt", + context=context, + ), from_email=settings.MAIL_DATA["revente"]["FROM"], to=[winner.user.email], ) ) - mails.append( - c_mails["bda-revente-seller"].get_message( - context, + EmailMessage( + subject=subject, + body=loader.render_to_string( + "bda/mails/revente-tirage-seller.txt", + context=context, + ), from_email=settings.MAIL_DATA["revente"]["FROM"], to=[seller.user.email], reply_to=[winner.user.email], - ) + ), ) # Envoie un mail aux perdants @@ -452,11 +460,15 @@ class SpectacleRevente(models.Model): new_context["acheteur"] = inscrit.user mails.append( - c_mails["bda-revente-loser"].get_message( - new_context, + EmailMessage( + subject=subject, + body=loader.render_to_string( + "bda/mails/revente-tirage-loser.txt", + context=new_context, + ), from_email=settings.MAIL_DATA["revente"]["FROM"], to=[inscrit.user.email], - ) + ), ) mail_conn = mail.get_connection() diff --git a/bda/templates/bda/mails-rappel.html b/bda/templates/bda/mails-rappel.html index c10503b0..c0770e47 100644 --- a/bda/templates/bda/mails-rappel.html +++ b/bda/templates/bda/mails-rappel.html @@ -26,13 +26,6 @@
-

- Note : le template de ce mail peut être modifié à - cette adresse -

- -
-

Forme des mails

Une seule place

diff --git a/bda/templates/bda/mails/attributions-decus.txt b/bda/templates/bda/mails/attributions-decus.txt new file mode 100644 index 00000000..69fadff6 --- /dev/null +++ b/bda/templates/bda/mails/attributions-decus.txt @@ -0,0 +1,10 @@ +Cher-e {{ member.first_name }}, + +Tu t'es inscrit-e pour le tirage au sort du BdA. Malheureusement, tu n'as +obtenu aucune place. + +Nous proposons cependant de nombreuses offres hors-tirage tout au long de +l'année, et nous t'invitons à nous contacter si l'une d'entre elles +t'intéresse ! +-- +Le Bureau des Arts \ No newline at end of file diff --git a/bda/templates/bda/mails/attributions.txt b/bda/templates/bda/mails/attributions.txt new file mode 100644 index 00000000..3fd10032 --- /dev/null +++ b/bda/templates/bda/mails/attributions.txt @@ -0,0 +1,31 @@ +Cher-e {{ member.first_name }}, + +Tu t'es inscrit-e pour le tirage au sort du BdA. Tu as été sélectionné-e +pour les spectacles suivants : +{% for place in places %} +- 1 place pour {{ place }}{% endfor %} + +*Paiement* +L'intégralité de ces places de spectacles est à régler dès maintenant et AVANT +vendredi prochain, au bureau du COF pendant les heures de permanences (du lundi au vendredi +entre 12h et 14h, et entre 18h et 20h). Des facilités de paiement sont bien +évidemment possibles : nous pouvons ne pas encaisser le chèque immédiatement, +ou bien découper votre paiement en deux fois. Pour ceux qui ne pourraient pas +venir payer au bureau, merci de nous contacter par mail. + +*Mode de retrait des places* +Au moment du paiement, certaines places vous seront remises directement, +d'autres seront à récupérer au cours de l'année, d'autres encore seront +nominatives et à retirer le soir même dans les théâtres correspondants. +Pour chaque spectacle, vous recevrez un mail quelques jours avant la +représentation vous indiquant le mode de retrait. + +Nous vous rappelons que l'obtention de places du BdA vous engage à +respecter les règles de fonctionnement : +https://bda.ens.fr/lequipe/charte-bda/ +Un système de revente des places via les mails BdA-revente est disponible +directement sur votre compte GestioCOF. + +En vous souhaitant de très beaux spectacles tout au long de l'année, +-- +Le Bureau des Arts \ No newline at end of file diff --git a/bda/templates/bda/mails/rappel.txt b/bda/templates/bda/mails/rappel.txt new file mode 100644 index 00000000..74614cbb --- /dev/null +++ b/bda/templates/bda/mails/rappel.txt @@ -0,0 +1,23 @@ +Bonjour {{ member.first_name }}, + +Nous te rappellons que tu as eu la chance d'obtenir {{ nb_attr|pluralize:"une place,deux places" }} +pour {{ show.title }}, le {{ show.date }} au {{ show.location }}. N'oublie pas de t'y rendre ! +{% if nb_attr == 2 %} +Tu as obtenu deux places pour ce spectacle. Nous te rappelons que +ces places sont strictement réservées aux personnes de moins de 28 ans. +{% endif %} +{% if show.listing %}Pour ce spectacle, tu as reçu {{ nb_attr|pluralize:"une place,des places" }} sur +listing. Il te faudra donc te rendre 15 minutes en avance sur les lieux de la représentation +pour {{ nb_attr|pluralize:"la,les" }} retirer. +{% else %}Pour assister à ce spectacle, tu dois présenter les billets qui ont +été distribués au burô. +{% endif %} + +Si tu ne peux plus assister à cette représentation, tu peux +revendre ta place via BdA-revente, accessible directement sur +GestioCOF (lien "revendre une place du premier tirage" sur la page +d'accueil https://www.cof.ens.fr/gestion/). + +En te souhaitant un excellent spectacle, +-- +Le Bureau des Arts \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-new.txt b/bda/templates/bda/mails/revente-new.txt new file mode 100644 index 00000000..7344011a --- /dev/null +++ b/bda/templates/bda/mails/revente-new.txt @@ -0,0 +1,12 @@ +Bonjour {{ member.first_name }} + +Une place pour le spectacle {{ show.title }} ({{ show.date }}) +a été postée sur BdA-Revente. + +Si ce spectacle t'intéresse toujours, merci de nous le signaler en cliquant +sur ce lien : https://{{ site }}{% url "bda-revente-confirm" revente.id %}. +Dans le cas où plusieurs personnes seraient intéressées, nous procèderons à +un tirage au sort le {{ revente.date_tirage|date:"DATE_FORMAT" }}. + +Chaleureusement, +Le BdA \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-seller.txt b/bda/templates/bda/mails/revente-seller.txt new file mode 100644 index 00000000..851ac09c --- /dev/null +++ b/bda/templates/bda/mails/revente-seller.txt @@ -0,0 +1,13 @@ +Bonjour {{ vendeur.first_name }}, + +Tu t’es bien inscrit·e pour revendre une place pour {{ show.title }}. + +{% with revente.date_tirage as time %} +Le tirage au sort entre tout·e·s les racheteuse·eur·s potentiel·le·s aura lieu +le {{ time|date:"DATE_FORMAT" }} à {{ time|time:"TIME_FORMAT" }} (dans {{time|timeuntil }}). +Si personne ne s’est inscrit pour racheter la place, celle-ci apparaîtra parmi +les « Places disponibles immédiatement à la revente » sur GestioCOF. +{% endwith %} + +Bonne revente ! +Le Bureau des Arts \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-shotgun-seller.txt b/bda/templates/bda/mails/revente-shotgun-seller.txt new file mode 100644 index 00000000..e67083fc --- /dev/null +++ b/bda/templates/bda/mails/revente-shotgun-seller.txt @@ -0,0 +1,6 @@ +Bonjour {{ vendeur.first_name }} ! + +Je souhaiterais racheter ta place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) à {{ show.price|floatformat:2 }}€. +Contacte-moi si tu es toujours intéressé·e ! + +{{ acheteur.get_full_name }} ({{ acheteur.email }}) \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-shotgun.txt b/bda/templates/bda/mails/revente-shotgun.txt new file mode 100644 index 00000000..e7b1ce29 --- /dev/null +++ b/bda/templates/bda/mails/revente-shotgun.txt @@ -0,0 +1,11 @@ +Bonjour {{ member.first_name }} + +Une place pour le spectacle {{ show.title }} ({{ show.date }}) +a été postée sur BdA-Revente. + +Puisque ce spectacle a lieu dans moins de 24h, il n'y a pas de tirage au sort pour +cette place : elle est disponible immédiatement à l'adresse +https://{{ site }}{% url "bda-revente-buy" show.id %}, à la disposition de tous. + +Chaleureusement, +Le BdA \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-tirage-loser.txt b/bda/templates/bda/mails/revente-tirage-loser.txt new file mode 100644 index 00000000..c1d49a01 --- /dev/null +++ b/bda/templates/bda/mails/revente-tirage-loser.txt @@ -0,0 +1,9 @@ +Bonjour {{ acheteur.first_name }}, + +Tu t'étais inscrit·e pour la revente de la place de {{ vendeur.get_full_name }} +pour {{ show.title }}. +Malheureusement, une autre personne a été tirée au sort pour racheter la place. +Tu pourras certainement retenter ta chance pour une autre revente ! + +À très bientôt, +Le Bureau des Arts \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-tirage-seller.txt b/bda/templates/bda/mails/revente-tirage-seller.txt new file mode 100644 index 00000000..7abff7ca --- /dev/null +++ b/bda/templates/bda/mails/revente-tirage-seller.txt @@ -0,0 +1,7 @@ +Bonjour {{ vendeur.first_name }}, + +La personne tirée au sort pour racheter ta place pour {{ show.title }} est {{ acheteur.get_full_name }}. +Tu peux le/la contacter à l'adresse {{ acheteur.email }}, ou en répondant à ce mail. + +Chaleureusement, +Le BdA \ No newline at end of file diff --git a/bda/templates/bda/mails/revente-tirage-winner.txt b/bda/templates/bda/mails/revente-tirage-winner.txt new file mode 100644 index 00000000..11428ef7 --- /dev/null +++ b/bda/templates/bda/mails/revente-tirage-winner.txt @@ -0,0 +1,7 @@ +Bonjour {{ acheteur.first_name }}, + +Tu as été tiré·e au sort pour racheter une place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) à {{ show.price|floatformat:2 }}€. +Tu peux contacter le/la vendeur·se à l'adresse {{ vendeur.email }}. + +Chaleureusement, +Le BdA \ No newline at end of file diff --git a/bda/tests/mixins.py b/bda/tests/mixins.py index a4ba057b..1f690172 100644 --- a/bda/tests/mixins.py +++ b/bda/tests/mixins.py @@ -1,7 +1,3 @@ -import os - -from django.conf import settings -from django.core.management import call_command from django.utils import timezone from shared.tests.mixins import ViewTestCaseMixin @@ -28,12 +24,6 @@ class BdATestHelpers: if self.bda_testdata: self.load_bda_testdata() - def require_custommails(self): - data_file = os.path.join( - settings.BASE_DIR, "gestioncof", "management", "data", "custommail.json" - ) - call_command("syncmails", data_file, verbosity=0) - def load_bda_testdata(self): self.tirage = Tirage.objects.create( title="Test tirage", diff --git a/bda/tests/test_models.py b/bda/tests/test_models.py index ba96238a..abff407b 100644 --- a/bda/tests/test_models.py +++ b/bda/tests/test_models.py @@ -19,8 +19,6 @@ User = get_user_model() class SpectacleReventeTests(TestCase): - fixtures = ["gestioncof/management/data/custommail.json"] - def setUp(self): now = timezone.now() diff --git a/bda/tests/test_views.py b/bda/tests/test_views.py index b223203f..75d01ec9 100644 --- a/bda/tests/test_views.py +++ b/bda/tests/test_views.py @@ -279,7 +279,6 @@ class SendRemindersViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase): return "/gestion/bda/mails-rappel/{}".format(self.show1.id) def test_post(self): - self.require_custommails() resp = self.client.post(self.url) self.assertEqual(200, resp.status_code) # TODO: check that emails are sent diff --git a/bda/views.py b/bda/views.py index 25fa97e2..1c54134e 100644 --- a/bda/views.py +++ b/bda/views.py @@ -4,17 +4,17 @@ import random import time from collections import defaultdict -from custommail.models import CustomMail -from custommail.shortcuts import send_custom_mail, send_mass_custom_mail from django.conf import settings from django.contrib import messages from django.core import serializers from django.core.exceptions import NON_FIELD_ERRORS +from django.core.mail import send_mail, send_mass_mail from django.db import transaction from django.db.models import Count, Prefetch from django.forms.models import inlineformset_factory from django.http import HttpResponseBadRequest, HttpResponseRedirect, JsonResponse from django.shortcuts import get_object_or_404, render +from django.template import loader from django.template.defaultfilters import pluralize from django.urls import reverse from django.utils import formats, timezone @@ -400,7 +400,7 @@ def revente_manage(request, tirage_id): if "resell" in request.POST: resellform = ResellForm(participant, request.POST, prefix="resell") if resellform.is_valid(): - datatuple = [] + mails = [] attributions = resellform.cleaned_data["attributions"] with transaction.atomic(): for attribution in attributions: @@ -415,16 +415,17 @@ def revente_manage(request, tirage_id): "show": attribution.spectacle, "revente": revente, } - datatuple.append( + mails.append( ( - "bda-revente-new", - context, + "BdA-Revente : {}".format(attribution.spectacle), + loader.render_to_string( + "bda/mails/revente-seller.txt", context=context + ), settings.MAIL_DATA["revente"]["FROM"], [participant.user.email], ) ) - revente.save() - send_mass_custom_mail(datatuple) + send_mass_mail(mails) # On annule une revente elif "annul" in request.POST: annulform = AnnulForm(participant, request.POST, prefix="annul") @@ -643,12 +644,16 @@ def revente_buy(request, spectacle_id): "acheteur": request.user, "vendeur": revente.seller.user, } - send_custom_mail( - "bda-buy-shotgun", - "bda@ens.fr", + + send_mail( + "BdA-Revente : {}".format(spectacle.title), + loader.render_to_string( + "bda/mails/revente-shotgun-seller.txt", context=context + ), + request.user.email, [revente.seller.user.email], - context=context, ) + return render( request, "bda/revente/mail-success.html", @@ -751,19 +756,21 @@ class UnpaidParticipants(BuroRequiredMixin, ListView): def send_rappel(request, spectacle_id): show = get_object_or_404(Spectacle, id=spectacle_id) # Mails d'exemples - custommail = CustomMail.objects.get(shortname="bda-rappel") - exemple_mail_1place = custommail.render( - {"member": request.user, "show": show, "nb_attr": 1} + subject = show.title + body_mail_1place = loader.render_to_string( + "bda/mails/rappel.txt", + context={"member": request.user, "show": show, "nb_attr": 1}, ) - exemple_mail_2places = custommail.render( - {"member": request.user, "show": show, "nb_attr": 2} + body_mail_2places = loader.render_to_string( + "bda/mails/rappel.txt", + context={"member": request.user, "show": show, "nb_attr": 2}, ) + # Contexte ctxt = { "show": show, - "exemple_mail_1place": exemple_mail_1place, - "exemple_mail_2places": exemple_mail_2places, - "custommail": custommail, + "exemple_mail_1place": (subject, body_mail_1place), + "exemple_mail_2places": (subject, body_mail_2places), } # Envoi confirmé if request.method == "POST": diff --git a/cof/settings/cof_prod.py b/cof/settings/cof_prod.py index 0f73841e..47fa3954 100644 --- a/cof/settings/cof_prod.py +++ b/cof/settings/cof_prod.py @@ -51,7 +51,6 @@ INSTALLED_APPS = ( "kfet", "kfet.open", "channels", - "custommail", "djconfig", "wagtail.contrib.forms", "wagtail.contrib.redirects", diff --git a/gestioncof/management/data/custommail.json b/gestioncof/management/data/custommail.json deleted file mode 100644 index 691043e9..00000000 --- a/gestioncof/management/data/custommail.json +++ /dev/null @@ -1,600 +0,0 @@ -[ -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "auth", - "user" - ], - "inner1": null, - "inner2": null - }, - "pk": 1 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "int", - "content_type": null, - "inner1": null, - "inner2": null - }, - "pk": 2 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "bda", - "spectacle" - ], - "inner1": null, - "inner2": null - }, - "pk": 3 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "bda", - "spectaclerevente" - ], - "inner1": null, - "inner2": null - }, - "pk": 4 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "sites", - "site" - ], - "inner1": null, - "inner2": null - }, - "pk": 5 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "gestioncof", - "petitcoursdemande" - ], - "inner1": null, - "inner2": null - }, - "pk": 6 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "list", - "content_type": null, - "inner1": 12, - "inner2": null - }, - "pk": 7 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "list", - "content_type": null, - "inner1": 1, - "inner2": null - }, - "pk": 8 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "pair", - "content_type": null, - "inner1": 12, - "inner2": 8 - }, - "pk": 9 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "list", - "content_type": null, - "inner1": 9, - "inner2": null - }, - "pk": 10 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "list", - "content_type": null, - "inner1": 3, - "inner2": null - }, - "pk": 11 -}, -{ - "model": "custommail.type", - "fields": { - "kind": "model", - "content_type": [ - "gestioncof", - "petitcourssubject" - ], - "inner1": null, - "inner2": null - }, - "pk": 12 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "welcome", - "subject": "Bienvenue au COF", - "body": "Bonjour {{ member.first_name }} et bienvenue au COF !\r\n\r\nTu trouveras plein de trucs cool sur le site du COF : https://www.cof.ens.fr/ et notre page Facebook : https://www.facebook.com/cof.ulm\r\nEt n'oublie pas d'aller d\u00e9couvrir GestioCOF, la plateforme de gestion du COF !\r\nSi tu as des questions, tu peux nous envoyer un mail \u00e0 cof@ens.fr (on aime le spam), ou passer nous voir au Bur\u00f4 pr\u00e8s de la Cour\u00f4 du lundi au vendredi de 12h \u00e0 14h et de 18h \u00e0 20h.\r\n\r\nRetrouvez les \u00e9v\u00e8nements de rentr\u00e9e pour les conscrit.e.s et les vieux/vieilles organis\u00e9s par le COF et ses clubs ici : http://www.cof.ens.fr/depot/Rentree.pdf \r\n\r\nAmicalement,\r\n\r\nTon COF qui t'aime.", - "description": "Mail de bienvenue au COF envoy\u00e9 automatiquement \u00e0 l'inscription d'un nouveau membre" - }, - "pk": 1 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-rappel", - "subject": "{{ show }}", - "body": "Bonjour {{ member.first_name }},\r\n\r\nNous te rappellons que tu as eu la chance d'obtenir {{ nb_attr|pluralize:\"une place,deux places\" }}\r\npour {{ show.title }}, le {{ show.date }} au {{ show.location }}. N'oublie pas de t'y rendre !\r\n{% if nb_attr == 2 %}\r\nTu as obtenu deux places pour ce spectacle. Nous te rappelons que\r\nces places sont strictement r\u00e9serv\u00e9es aux personnes de moins de 28 ans.\r\n{% endif %}\r\n{% if show.listing %}Pour ce spectacle, tu as re\u00e7u des places sur\r\nlisting. Il te faudra donc te rendre 15 minutes en avance sur les lieux de la repr\u00e9sentation\r\npour retirer {{ nb_attr|pluralize:\"ta place,tes places\" }}.\r\n{% else %}Pour assister \u00e0 ce spectacle, tu dois pr\u00e9senter les billets qui ont\r\n\u00e9t\u00e9 distribu\u00e9s au bur\u00f4.\r\n{% endif %}\r\n\r\nSi tu ne peux plus assister \u00e0 cette repr\u00e9sentation, tu peux\r\nrevendre ta place via BdA-revente, accessible directement sur\r\nGestioCOF (lien \"revendre une place du premier tirage\" sur la page\r\nd'accueil https://www.cof.ens.fr/gestion/).\r\n\r\nEn te souhaitant un excellent spectacle,\r\n\r\nLe Bureau des Arts", - "description": "Mail de rappel pour les spectacles BdA" - }, - "pk": 2 -}, -{ - "model": "custommail.custommail", - "pk": 3, - "fields": { - "shortname": "bda-revente", - "subject": "{{ show }}", - "description": "Notification envoy\u00e9e \u00e0 toutes les personnes int\u00e9ress\u00e9es par un spectacle pour leur signaler qu'une place vient d'\u00eatre mise en vente.", - "body": "Bonjour {{ member.first_name }}\r\n\r\nUne place pour le spectacle {{ show.title }} ({{ show.date }})\r\na \u00e9t\u00e9 post\u00e9e sur BdA-Revente.\r\n\r\nSi ce spectacle t'int\u00e9resse toujours, merci de nous le signaler en cliquant\r\nsur ce lien : http://{{ site }}{% url \"bda-revente-confirm\" revente.id %}.\r\nDans le cas o\u00f9 plusieurs personnes seraient int\u00e9ress\u00e9es, nous proc\u00e8derons \u00e0\r\nun tirage au sort le {{ revente.date_tirage|date:\"DATE_FORMAT\" }}.\r\n\r\nChaleureusement,\r\nLe BdA" - } -}, -{ - "model": "custommail.custommail", - "pk": 4, - "fields": { - "shortname": "bda-shotgun", - "subject": "{{ show }}", - "description": "Notification signalant qu'une place est au shotgun aux personnes int\u00e9ress\u00e9es.", - "body": "Bonjour {{ member.first_name }}\r\n\r\nUne place pour le spectacle {{ show.title }} ({{ show.date }})\r\na \u00e9t\u00e9 post\u00e9e sur BdA-Revente.\r\n\r\nPuisque ce spectacle a lieu dans moins de 24h, il n'y a pas de tirage au sort pour\r\ncette place : elle est disponible imm\u00e9diatement \u00e0 l'adresse\r\nhttp://{{ site }}{% url \"bda-revente-buy\" show.id %}, \u00e0 la disposition de tous.\r\n\r\nChaleureusement,\r\nLe BdA" - } -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-revente-winner", - "subject": "BdA-Revente : {{ show.title }}", - "body": "Bonjour {{ acheteur.first_name }},\r\n\r\nTu as \u00e9t\u00e9 tir\u00e9-e au sort pour racheter une place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) \u00e0 {{ show.price|floatformat:2 }}\u20ac.\r\nTu peux contacter le/la vendeur-se \u00e0 l'adresse {{ vendeur.email }}.\r\n\r\nChaleureusement,\r\nLe BdA", - "description": "Mail envoy\u00e9 au gagnant d'un tirage BdA-Revente" - }, - "pk": 5 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-revente-loser", - "subject": "BdA-Revente : {{ show.title }}", - "body": "Bonjour {{ acheteur.first_name }},\r\n\r\nTu t'\u00e9tais inscrit-e pour la revente de la place de {{ vendeur.get_full_name }}\r\npour {{ show.title }}.\r\nMalheureusement, une autre personne a \u00e9t\u00e9 tir\u00e9e au sort pour racheter la place.\r\nTu pourras certainement retenter ta chance pour une autre revente !\r\n\r\n\u00c0 tr\u00e8s bient\u00f4t,\r\nLe Bureau des Arts", - "description": "Notification envoy\u00e9e aux perdants d'un tirage de revente." - }, - "pk": 6 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-revente-seller", - "subject": "BdA-Revente : {{ show.title }}", - "body": "Bonjour {{ vendeur.first_name }},\r\n\r\nLa personne tir\u00e9e au sort pour racheter ta place pour {{ show.title }} est {{ acheteur.get_full_name }}.\r\nTu peux le/la contacter \u00e0 l'adresse {{ acheteur.email }}, ou en r\u00e9pondant \u00e0 ce mail.\r\n\r\nChaleureusement,\r\nLe BdA", - "description": "Notification envoy\u00e9e au vendeur d'une place pour lui indiquer qu'elle vient d'\u00eatre attribu\u00e9e" - }, - "pk": 7 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-revente-new", - "subject": "BdA-Revente : {{ show.title }}", - "body": "Bonjour {{ vendeur.first_name }},\r\n\r\nTu t\u2019es bien inscrit-e pour la revente de {{ show.title }}.\r\n\r\n{% with revente.date_tirage as time %}\r\nLe tirage au sort entre tout-e-s les racheteuse-eur-s potentiel-le-s aura lieu\r\nle {{ time|date:\"DATE_FORMAT\" }} \u00e0 {{ time|time:\"TIME_FORMAT\" }} (dans {{time|timeuntil }}).\r\nSi personne ne s\u2019est inscrit pour racheter la place, celle-ci apparaitra parmi\r\nles \u00ab Places disponibles imm\u00e9diatement \u00e0 la revente \u00bb sur GestioCOF.\r\n{% endwith %}\r\n\r\nBonne revente !\r\nLe Bureau des Arts", - "description": "Notification signalant au vendeur d'une place que sa mise en vente a bien eu lieu et lui donnant quelques informations compl\u00e9mentaires." - }, - "pk": 8 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-buy-shotgun", - "subject": "BdA-Revente : {{ show.title }}", - "body": "Bonjour {{ vendeur.first_name }} !\r\n\r\nJe souhaiterais racheter ta place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) \u00e0 {{ show.price|floatformat:2 }}\u20ac.\r\nContacte-moi si tu es toujours int\u00e9ress\u00e9\u00b7e !\r\n\r\n{{ acheteur.get_full_name }} ({{ acheteur.email }})", - "description": "Mail envoy\u00e9 au revendeur lors d'un achat au shotgun." - }, - "pk": 9 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "petit-cours-mail-eleve", - "subject": "Petits cours ENS par le COF", - "body": "Salut,\r\n\r\nLe COF a re\u00e7u une demande de petit cours qui te correspond. Tu es en haut de la liste d'attente donc on a transmis tes coordonn\u00e9es, ainsi que celles de 2 autres qui correspondaient aussi (c'est la vie, on donne les num\u00e9ros 3 par 3 pour que ce soit plus souple). Voici quelques infos sur l'annonce en question :\r\n\r\n\u00a4 Nom : {{ demande.name }}\r\n\r\n\u00a4 P\u00e9riode : {{ demande.quand }}\r\n\r\n\u00a4 Fr\u00e9quence : {{ demande.freq }}\r\n\r\n\u00a4 Lieu (si pr\u00e9f\u00e9r\u00e9) : {{ demande.lieu }}\r\n\r\n\u00a4 Niveau : {{ demande.get_niveau_display }}\r\n\r\n\u00a4 Remarques diverses (d\u00e9sol\u00e9 pour les balises HTML) : {{ demande.remarques }}\r\n\r\n{% if matieres|length > 1 %}\u00a4 Mati\u00e8res :\r\n{% for matiere in matieres %} \u00a4 {{ matiere }}\r\n{% endfor %}{% else %}\u00a4 Mati\u00e8re : {% for matiere in matieres %}{{ matiere }}\r\n{% endfor %}{% endif %}\r\nVoil\u00e0, cette personne te contactera peut-\u00eatre sous peu, tu pourras voir les d\u00e9tails directement avec elle (prix, modalit\u00e9s, ...). Pour indication, 30 Euro/h semble \u00eatre la moyenne.\r\n\r\nSi tu te rends compte qu'en fait tu ne peux pas/plus donner de cours en ce moment, \u00e7a serait cool que tu d\u00e9coches la case \"Recevoir des propositions de petits cours\" sur GestioCOF. Ensuite d\u00e8s que tu voudras r\u00e9appara\u00eetre tu pourras recocher la case et tu seras \u00e0 nouveau sur la liste.\r\n\r\n\u00c0 bient\u00f4t,\r\n\r\n--\r\nLe COF, pour les petits cours", - "description": "Mail envoy\u00e9 aux personnes dont ont a donn\u00e9 les contacts \u00e0 des demandeurs de petits cours" - }, - "pk": 10 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "petits-cours-mail-demandeur", - "subject": "Cours particuliers ENS", - "body": "Bonjour,\r\n\r\nJe vous contacte au sujet de votre annonce pass\u00e9e sur le site du COF pour rentrer en contact avec un \u00e9l\u00e8ve normalien pour des cours particuliers. Voici les coordonn\u00e9es d'\u00e9l\u00e8ves qui sont motiv\u00e9s par de tels cours et correspondent aux crit\u00e8res que vous nous aviez transmis :\r\n\r\n{% for matiere, proposed in proposals %}\u00a4 {{ matiere }} :{% for user in proposed %}\r\n \u00a4 {{ user.get_full_name }}{% if user.profile.phone %}, {{ user.profile.phone }}{% endif %}{% if user.email %}, {{ user.email }}{% endif %}{% endfor %}\r\n\r\n{% endfor %}{% if unsatisfied %}Nous n'avons cependant pas pu trouver d'\u00e9l\u00e8ve disponible pour des cours de {% for matiere in unsatisfied %}{% if forloop.counter0 > 0 %}, {% endif %}{{ matiere }}{% endfor %}.\r\n\r\n{% endif %}Si pour une raison ou une autre ces num\u00e9ros ne suffisaient pas, n'h\u00e9sitez pas \u00e0 r\u00e9pondre \u00e0 cet e-mail et je vous en ferai parvenir d'autres sans probl\u00e8me.\r\n{% if extra|length > 0 %}\r\n{{ extra|safe }}\r\n{% endif %}\r\nCordialement,\r\n\r\n--\r\nLe COF, BdE de l'ENS", - "description": "Mail envoy\u00e9 aux personnes qui demandent des petits cours lorsque leur demande est trait\u00e9e.\r\n\r\n(Ne pas toucher \u00e0 {{ extra|safe }})" - }, - "pk": 11 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-attributions", - "subject": "R\u00e9sultats du tirage au sort", - "body": "Cher-e {{ member.first_name }},\r\n\r\nTu t'es inscrit-e pour le tirage au sort du BdA. Tu as \u00e9t\u00e9 s\u00e9lectionn\u00e9-e\r\npour les spectacles suivants :\r\n{% for place in places %}\r\n- 1 place pour {{ place }}{% endfor %}\r\n\r\n*Paiement*\r\nL'int\u00e9gralit\u00e9 de ces places de spectacles est \u00e0 r\u00e9gler d\u00e8s maintenant et AVANT\r\nvendredi prochain, au bureau du COF pendant les heures de permanences (du lundi au vendredi\r\nentre 12h et 14h, et entre 18h et 20h). Des facilit\u00e9s de paiement sont bien\r\n\u00e9videmment possibles : nous pouvons ne pas encaisser le ch\u00e8que imm\u00e9diatement,\r\nou bien d\u00e9couper votre paiement en deux fois. Pour ceux qui ne pourraient pas\r\nvenir payer au bureau, merci de nous contacter par mail.\r\n\r\n*Mode de retrait des places*\r\nAu moment du paiement, certaines places vous seront remises directement,\r\nd'autres seront \u00e0 r\u00e9cup\u00e9rer au cours de l'ann\u00e9e, d'autres encore seront\r\nnominatives et \u00e0 retirer le soir m\u00eame dans les the\u00e2tres correspondants.\r\nPour chaque spectacle, vous recevrez un mail quelques jours avant la\r\nrepr\u00e9sentation vous indiquant le mode de retrait.\r\n\r\nNous vous rappelons que l'obtention de places du BdA vous engage \u00e0\r\nrespecter les r\u00e8gles de fonctionnement :\r\nhttp://www.cof.ens.fr/bda/?page_id=1370\r\nUn syst\u00e8me de revente des places via les mails BdA-revente disponible\r\ndirectement sur votre compte GestioCOF.\r\n\r\nEn vous souhaitant de tr\u00e8s beaux spectacles tout au long de l'ann\u00e9e,\r\n--\r\nLe Bureau des Arts", - "description": "Mail annon\u00e7ant les r\u00e9sultats du tirage au sort du BdA aux gagnants d'une ou plusieurs places" - }, - "pk": 12 -}, -{ - "model": "custommail.custommail", - "fields": { - "shortname": "bda-attributions-decus", - "subject": "R\u00e9sultats du tirage au sort", - "body": "Cher-e {{ member.first_name }},\r\n\r\nTu t'es inscrit-e pour le tirage au sort du BdA. Malheureusement, tu n'as\r\nobtenu aucune place.\r\n\r\nNous proposons cependant de nombreuses offres hors-tirage tout au long de\r\nl'ann\u00e9e, et nous t'invitons \u00e0 nous contacter si l'une d'entre elles\r\nt'int\u00e9resse !\r\n--\r\nLe Bureau des Arts", - "description": "Mail annon\u00e7ant les r\u00e9sultats du tirage au sort du BdA aux personnes n'ayant pas obtenu de place" - }, - "pk": 13 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 1, - "type": 1, - "name": "member", - "description": "Utilisateur de GestioCOF" - }, - "pk": 1 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 2, - "type": 1, - "name": "member", - "description": "Utilisateur ayant eu une place pour ce spectacle" - }, - "pk": 2 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 2, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 3 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 2, - "type": 2, - "name": "nb_attr", - "description": "Nombre de places obtenues" - }, - "pk": 4 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 3, - "type": 4, - "name": "revente", - "description": "Revente mentionn\u00e9e dans le mail" - }, - "pk": 5 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 3, - "type": 1, - "name": "member", - "description": "Personne int\u00e9ress\u00e9e par la place" - }, - "pk": 6 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 3, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 7 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 3, - "type": 5, - "name": "site", - "description": "Site web (gestioCOF)" - }, - "pk": 8 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 4, - "type": 5, - "name": "site", - "description": "Site web (gestioCOF)" - }, - "pk": 9 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 4, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 10 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 4, - "type": 1, - "name": "member", - "description": "Personne int\u00e9ress\u00e9e par la place" - }, - "pk": 11 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 5, - "type": 1, - "name": "acheteur", - "description": "Gagnant-e du tirage" - }, - "pk": 12 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 5, - "type": 1, - "name": "vendeur", - "description": "Personne qui vend une place" - }, - "pk": 13 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 5, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 14 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 6, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 15 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 6, - "type": 1, - "name": "vendeur", - "description": "Personne qui vend une place" - }, - "pk": 16 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 6, - "type": 1, - "name": "acheteur", - "description": "Personne inscrite au tirage qui n'a pas eu la place" - }, - "pk": 17 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 7, - "type": 1, - "name": "acheteur", - "description": "Gagnant-e du tirage" - }, - "pk": 18 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 7, - "type": 1, - "name": "vendeur", - "description": "Personne qui vend une place" - }, - "pk": 19 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 7, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 20 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 8, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 21 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 8, - "type": 1, - "name": "vendeur", - "description": "Personne qui vend la place" - }, - "pk": 22 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 8, - "type": 4, - "name": "revente", - "description": "Revente mentionn\u00e9e dans le mail" - }, - "pk": 23 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 9, - "type": 1, - "name": "vendeur", - "description": "Personne qui vend la place" - }, - "pk": 24 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 9, - "type": 3, - "name": "show", - "description": "Spectacle" - }, - "pk": 25 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 9, - "type": 1, - "name": "acheteur", - "description": "Personne qui prend la place au shotgun" - }, - "pk": 26 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 10, - "type": 6, - "name": "demande", - "description": "Demande de petit cours" - }, - "pk": 27 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 10, - "type": 7, - "name": "matieres", - "description": "Liste des mati\u00e8res concern\u00e9es par la demande" - }, - "pk": 28 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 11, - "type": 10, - "name": "proposals", - "description": "Liste associant une liste d'enseignants \u00e0 chaque mati\u00e8re" - }, - "pk": 29 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 11, - "type": 7, - "name": "unsatisfied", - "description": "Liste des mati\u00e8res pour lesquelles on n'a pas d'enseigant \u00e0 proposer" - }, - "pk": 30 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 12, - "type": 11, - "name": "places", - "description": "Places de spectacle du participant" - }, - "pk": 31 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 12, - "type": 1, - "name": "member", - "description": "Participant du tirage au sort" - }, - "pk": 32 -}, -{ - "model": "custommail.variable", - "fields": { - "custommail": 13, - "type": 1, - "name": "member", - "description": "Participant du tirage au sort" - }, - "pk": 33 -} -] diff --git a/gestioncof/templates/gestioncof/mails/welcome.txt b/gestioncof/templates/gestioncof/mails/welcome.txt new file mode 100644 index 00000000..aa7ff326 --- /dev/null +++ b/gestioncof/templates/gestioncof/mails/welcome.txt @@ -0,0 +1,11 @@ +Bonjour {{ member.first_name }} et bienvenue au COF ! + +Tu trouveras plein de trucs cool sur le site du COF : https://cof.ens.fr/ et notre page Facebook : https://www.facebook.com/cof.ulm +Et n'oublie pas d'aller découvrir GestioCOF, la plateforme de gestion du COF ! +Si tu as des questions, tu peux nous envoyer un mail à cof@ens.fr (on aime le spam), ou passer nous voir au Burô près de la Courô du lundi au vendredi de 12h à 14h et de 18h à 20h. + +Retrouvez les évènements de rentrée pour les conscrit.e.s et les vieux/vieilles organisés par le COF et ses clubs ici : https://cof.ens.fr/planningrentree. + +Amicalement, + +Ton COF qui t'aime. \ No newline at end of file diff --git a/gestioncof/tests/test_views.py b/gestioncof/tests/test_views.py index ea51c509..dc9b8df0 100644 --- a/gestioncof/tests/test_views.py +++ b/gestioncof/tests/test_views.py @@ -1,15 +1,13 @@ -import os import uuid from datetime import timedelta -from custommail.models import CustomMail -from django.conf import settings from django.contrib import messages from django.contrib.auth import get_user_model from django.contrib.messages.api import get_messages from django.contrib.messages.storage.base import Message from django.core import mail -from django.core.management import call_command +from django.core.mail import EmailMessage +from django.template import loader from django.test import Client, TestCase, override_settings from django.urls import reverse @@ -33,12 +31,6 @@ class RegistrationViewTests(ViewTestCaseMixin, TestCase): auth_user = "staff" auth_forbidden = [None, "user", "member"] - def requires_mails(self): - data_file = os.path.join( - settings.BASE_DIR, "gestioncof", "management", "data", "custommail.json" - ) - call_command("syncmails", data_file, verbosity=0) - def test_get(self): r = self.client.get(self.url) self.assertEqual(r.status_code, 200) @@ -64,8 +56,6 @@ class RegistrationViewTests(ViewTestCaseMixin, TestCase): } def test_post_new(self): - self.requires_mails() - r = self.client.post( self.url, dict( @@ -97,7 +87,6 @@ class RegistrationViewTests(ViewTestCaseMixin, TestCase): self.assertEqual(u.email, "username@mail.net") def test_post_edit(self): - self.requires_mails() u = self.users["user"] r = self.client.post( @@ -132,7 +121,6 @@ class RegistrationViewTests(ViewTestCaseMixin, TestCase): self.assertEqual(u.email, "user@mail.net") def _test_mail_welcome(self, was_cof, is_cof, expect_mail): - self.requires_mails() u = self.users["member"] if was_cof else self.users["user"] data = dict( @@ -146,8 +134,12 @@ class RegistrationViewTests(ViewTestCaseMixin, TestCase): u.refresh_from_db() def _is_sent(): - cm = CustomMail.objects.get(shortname="welcome") - welcome_msg = cm.get_message({"member": u}) + welcome_msg = EmailMessage( + subject="Bienvenue au COF", + body=loader.render_to_string( + "gestioncof/mails/welcome.txt", context={"member": "u"} + ), + ) for m in mail.outbox: if m.subject == welcome_msg.subject: return True diff --git a/gestioncof/views.py b/gestioncof/views.py index 9c9dfd3c..d4b6a5be 100644 --- a/gestioncof/views.py +++ b/gestioncof/views.py @@ -4,7 +4,6 @@ from datetime import timedelta from smtplib import SMTPRecipientsRefused from urllib.parse import parse_qs, urlencode, urlparse, urlunparse -from custommail.shortcuts import send_custom_mail from django.contrib import messages from django.contrib.auth.decorators import login_required from django.contrib.auth.mixins import LoginRequiredMixin @@ -15,8 +14,10 @@ from django.contrib.auth.views import ( redirect_to_login, ) from django.contrib.sites.models import Site +from django.core.mail import send_mail from django.http import Http404, HttpResponse, HttpResponseForbidden from django.shortcuts import get_object_or_404, redirect, render +from django.template import loader from django.urls import reverse_lazy from django.utils import timezone from django.utils.translation import ugettext_lazy as _ @@ -489,8 +490,13 @@ def notify_new_member(request, member: User): # Try to send a welcome email and report SMTP errors try: - send_custom_mail( - "welcome", "cof@ens.fr", [member.email], context={"member": member} + send_mail( + "Bienvenue au COF", + loader.render_to_string( + "gestioncof/mails/welcome.txt", context={"member": member} + ), + "cof@ens.fr", + [member.email], ) except SMTPRecipientsRefused: messages.error( diff --git a/petitscours/templates/petitscours/mails/demandeur.txt b/petitscours/templates/petitscours/mails/demandeur.txt new file mode 100644 index 00000000..69fed436 --- /dev/null +++ b/petitscours/templates/petitscours/mails/demandeur.txt @@ -0,0 +1,17 @@ +Bonjour, + +Je vous contacte au sujet de votre annonce passée sur le site du COF pour rentrer en contact avec un élève normalien pour des cours particuliers. Voici les coordonnées d'élèves qui sont motivés par de tels cours et correspondent aux critères que vous nous aviez transmis : + +{% for matiere, proposed in proposals %}¤ {{ matiere }} :{% for user in proposed %} + ¤ {{ user.get_full_name }}{% if user.profile.phone %}, {{ user.profile.phone }}{% endif %}{% if user.email %}, {{ user.email }}{% endif %}{% endfor %} + +{% endfor %}{% if unsatisfied %}Nous n'avons cependant pas pu trouver d'élève disponible pour des cours de {% for matiere in unsatisfied %}{% if forloop.counter0 > 0 %}, {% endif %}{{ matiere }}{% endfor %}. + +{% endif %}Si pour une raison ou une autre ces numéros ne suffisaient pas, n'hésitez pas à répondre à cet e-mail et je vous en ferai parvenir d'autres sans problème. +{% if extra|length > 0 %} +{{ extra|safe }} +{% endif %} +Cordialement, + +-- +Le COF, BdE de l'ENS \ No newline at end of file diff --git a/petitscours/templates/petitscours/mails/eleve.txt b/petitscours/templates/petitscours/mails/eleve.txt new file mode 100644 index 00000000..1f8fe14d --- /dev/null +++ b/petitscours/templates/petitscours/mails/eleve.txt @@ -0,0 +1,28 @@ +Salut, + +Le COF a reçu une demande de petit cours qui te correspond. Tu es en haut de la liste d'attente donc on a transmis tes coordonnées, ainsi que celles de 2 autres qui correspondaient aussi (c'est la vie, on donne les numéros 3 par 3 pour que ce soit plus souple). Voici quelques infos sur l'annonce en question : + +¤ Nom : {{ demande.name }} + +¤ Période : {{ demande.quand }} + +¤ Fréquence : {{ demande.freq }} + +¤ Lieu (si préféré) : {{ demande.lieu }} + +¤ Niveau : {{ demande.get_niveau_display }} + +¤ Remarques diverses (désolé pour les balises HTML) : {{ demande.remarques }} + +{% if matieres|length > 1 %}¤ Matières : +{% for matiere in matieres %} ¤ {{ matiere }} +{% endfor %}{% else %}¤ Matière : {% for matiere in matieres %}{{ matiere }} +{% endfor %}{% endif %} +Voilà, cette personne te contactera peut-être sous peu, tu pourras voir les détails directement avec elle (prix, modalités, ...). Pour indication, 30 Euro/h semble être la moyenne. + +Si tu te rends compte qu'en fait tu ne peux pas/plus donner de cours en ce moment, ça serait cool que tu décoches la case "Recevoir des propositions de petits cours" sur GestioCOF. Ensuite dès que tu voudras réapparaître tu pourras recocher la case et tu seras à nouveau sur la liste. + +À bientôt, + +-- +Le COF, pour les petits cours \ No newline at end of file diff --git a/petitscours/tests/test_views.py b/petitscours/tests/test_views.py index 6cf520a2..aee1f2e8 100644 --- a/petitscours/tests/test_views.py +++ b/petitscours/tests/test_views.py @@ -8,7 +8,6 @@ from django.urls import reverse from gestioncof.tests.mixins import ViewTestCaseMixin from .utils import ( - PetitCoursTestHelpers, create_petitcours_ability, create_petitcours_demande, create_petitcours_subject, @@ -160,9 +159,7 @@ class PetitCoursInscriptionViewTestCase(ViewTestCaseMixin, TestCase): self.assertFalse(self.user.petitcoursability_set.all()) -class PetitCoursTraitementViewTestCase( - ViewTestCaseMixin, PetitCoursTestHelpers, TestCase -): +class PetitCoursTraitementViewTestCase(ViewTestCaseMixin, TestCase): url_name = "petits-cours-demande-traitement" http_methods = ["GET", "POST"] @@ -188,14 +185,10 @@ class PetitCoursTraitementViewTestCase( self.demande.matieres.add(self.subject) def test_get(self): - self.require_custommails() - resp = self.client.get(self.url) self.assertEqual(resp.status_code, 200) def test_get_with_match(self): - self.require_custommails() - create_petitcours_ability( user=self.user, matiere=self.subject, niveau="college" ) @@ -211,8 +204,6 @@ class PetitCoursTraitementViewTestCase( ) def test_post_with_match(self): - self.require_custommails() - create_petitcours_ability( user=self.user, matiere=self.subject, niveau="college" ) @@ -230,9 +221,7 @@ class PetitCoursTraitementViewTestCase( self.assertIsNotNone(self.demande.processed) -class PetitCoursRetraitementViewTestCase( - ViewTestCaseMixin, PetitCoursTestHelpers, TestCase -): +class PetitCoursRetraitementViewTestCase(ViewTestCaseMixin, TestCase): url_name = "petits-cours-demande-retraitement" http_methods = ["GET", "POST"] @@ -253,8 +242,6 @@ class PetitCoursRetraitementViewTestCase( self.demande = create_petitcours_demande() def test_get(self): - self.require_custommails() - resp = self.client.get(self.url) self.assertEqual(resp.status_code, 200) diff --git a/petitscours/tests/utils.py b/petitscours/tests/utils.py index 99113f9e..131f14dc 100644 --- a/petitscours/tests/utils.py +++ b/petitscours/tests/utils.py @@ -1,8 +1,4 @@ -import os - -from django.conf import settings -from django.core.management import call_command - +from gestioncof.tests.utils import create_user from petitscours.models import ( PetitCoursAbility, PetitCoursAttributionCounter, @@ -13,7 +9,7 @@ from petitscours.models import ( def create_petitcours_ability(**kwargs): if "user" not in kwargs: - kwargs["user"] = create_user() + kwargs["user"] = create_user("toto") if "matiere" not in kwargs: kwargs["matiere"] = create_petitcours_subject() if "niveau" not in kwargs: @@ -29,11 +25,3 @@ def create_petitcours_demande(**kwargs): def create_petitcours_subject(**kwargs): return PetitCoursSubject.objects.create(**kwargs) - - -class PetitCoursTestHelpers: - def require_custommails(self): - data_file = os.path.join( - settings.BASE_DIR, "gestioncof", "management", "data", "custommail.json" - ) - call_command("syncmails", data_file, verbosity=0) diff --git a/petitscours/views.py b/petitscours/views.py index e309f24a..81504e15 100644 --- a/petitscours/views.py +++ b/petitscours/views.py @@ -1,6 +1,5 @@ import json -from custommail.shortcuts import render_custom_mail from django.conf import settings from django.contrib import messages from django.contrib.auth.decorators import login_required @@ -8,6 +7,7 @@ from django.contrib.auth.models import User from django.core import mail from django.db import transaction from django.shortcuts import get_object_or_404, redirect, render +from django.template import loader from django.utils import timezone from django.views.decorators.csrf import csrf_exempt from django.views.generic import DetailView, ListView @@ -70,15 +70,18 @@ def _finalize_traitement( proposed_for.setdefault(user, []).append(matiere) proposed_mails = _generate_eleve_email(demande, proposed_for) - mainmail = render_custom_mail( - "petits-cours-mail-demandeur", - { - "proposals": proposals.items(), - "unsatisfied": unsatisfied, - "extra": '", - }, + mainmail = ( + "Cours particuliers ENS", + loader.render_to_string( + "petitscours/mails/demandeur.txt", + context={ + "proposals": proposals.items(), + "unsatisfied": unsatisfied, + "extra": '", + }, + ), ) if errors is not None: for error in errors: @@ -100,11 +103,16 @@ def _finalize_traitement( def _generate_eleve_email(demande, proposed_for): + subject = "Petits cours ENS par le COF" return [ ( user, - render_custom_mail( - "petit-cours-mail-eleve", {"demande": demande, "matieres": matieres} + ( + subject, + loader.render_to_string( + "petitscours/mails/eleve.txt", + context={"demande": demande, "matieres": matieres}, + ), ), ) for user, matieres in proposed_for.items() @@ -197,9 +205,14 @@ def _traitement_post(request, demande): else: proposed_for[user].append(matiere) proposed_mails = _generate_eleve_email(demande, proposed_for) - mainmail_object, mainmail_body = render_custom_mail( - "petits-cours-mail-demandeur", - {"proposals": proposals.items(), "unsatisfied": unsatisfied, "extra": extra}, + mainmail_object = "Cours particuliers ENS" + mainmail_body = loader.render_to_string( + "petitscours/mails/demandeur.txt", + context={ + "proposals": proposals.items(), + "unsatisfied": unsatisfied, + "extra": extra, + }, ) frommail = settings.MAIL_DATA["petits_cours"]["FROM"] bccaddress = settings.MAIL_DATA["petits_cours"]["BCC"] diff --git a/provisioning/prepare_django.sh b/provisioning/prepare_django.sh index f3358873..73745544 100644 --- a/provisioning/prepare_django.sh +++ b/provisioning/prepare_django.sh @@ -8,4 +8,3 @@ python manage.py sync_page_translation_fields python manage.py update_translation_fields python manage.py loaddata gestion sites articles python manage.py loaddevdata -python manage.py syncmails gestioncof/management/data/custommail.json diff --git a/requirements.txt b/requirements.txt index bf58d8f4..565d2b71 100644 --- a/requirements.txt +++ b/requirements.txt @@ -9,7 +9,6 @@ Pillow django-bootstrap-form==3.3 statistics==1.0.3.5 django-widget-tweaks==1.4.1 -git+https://git.eleves.ens.fr/cof-geek/django_custommail.git#egg=django_custommail channels==1.1.* python-dateutil wagtail==2.7.*