[FEATURE] Improve registration update mails (#786)

* Send email to user being promoted from waiting list

* Make it possible to count/test send emails + fix local/extranet tests

* Allow providing reason for rejection/unregistration. Send mail when confirmed from wairting list

* Test mails sent on unresgistration

* Add update_attendance test

* Linting

* Add registration deletion test

* Add test for waiting list promotion for upcoming event

Author: gdaviet

collectives/configuration.yaml

@@ -277,11 +277,13 @@ Template mail:
         description: Email subject for user self unregister
         type: ShortString
 
-    SELF_UNREGISTER_MESSAGE:
+    SELF_UNREGISTER_MESSAGE_v2:
         content: |
             Bonjour,
 
             '{user_name}' vient de se désinscrire de l'événement '{event_title}' que vous encadrez.
+            {reason}
+            
             Lien vers l'événement :
             {link}
 
@@ -400,11 +402,13 @@ Template mail:
         description: Email subject for rejected registration to an event
         type: ShortString
 
-    REJECTED_REGISTRATION_MESSAGE:
+    REJECTED_REGISTRATION_MESSAGE_v2:
         content: |
             Bonjour,
 
-            {rejector_name} vient de refuser votre inscription à la collective {event_title} débutant le {event_date}.
+            {rejector_name} vient de refuser votre inscription à l'événement {event_title} débutant le {event_date}.
+            {reason}
+            
             Lien vers l'événement :
             {link}
 
@@ -437,42 +441,35 @@ Template mail:
         description: Sujet de l'email pour l'utilisateur lorsque que son inscription est activée.
         type: ShortString
 
-    ACTIVATED_REGISTRATION_MESSAGE:
+    ACTIVATED_REGISTRATION_MESSAGE_v2:
         content: |
             Bonjour,
 
-            Une place vient de se libérer pour la collective {event_title} débutant le {event_date}.
-            Votre place dans la liste d'attente a donc été validée et vous êtes à présent inscrit(e)
-            à la collective, ou en attente de paiement si la collective en nécessite un.
-
-            Si vous ne pouvez plus participer, merci de vous désincrire au plus tôt afin de ne pas 
-            perturber l'organisation de la collective.
+            Votre inscription vient d'être validée pour l'événement {event_title} débutant le {event_date}
+            pour lequel vous étiez jusqu'alors en liste d'attente.
 
-            Merci de vérifier si la collective nécessite un paiement de votre part:
+            Merci de vérifier si l'événement nécessite un paiement de votre part et de procéder à celui-ci le cas échéant:
             {link}
 
+            {unregistration_text}
+
             Vous recevez cet e-mail en tant qu'adhérent inscrit à cette collective.
             Cet e-mail est envoyé par un automate, répondre à cet e-mail sera sans effet.
         description: Template d'email pour l'utilisateur lorsque que son inscription est activée.
         type: LongString
-
-    ACTIVATED_REGISTRATION_UPCOMING_EVENT_MESSAGE:
+    
+    ACTIVATED_REGISTRATION_STANDARD_EVENT_MESSAGE:
         content: |
-            Bonjour,
-
-            Une place vient de se libérer pour la collective {event_title} débutant le {event_date}.
-            Votre place dans la liste d'attente a donc été validée et vous êtes à présent inscrit(e)
-            à la collective, ou en attente de paiement si la collective en nécessite un.
+            Si vous ne pouvez plus participer, merci de vous désincrire au plus tôt afin de ne pas 
+            perturber l'organisation de la collective.
+        description: Template d'email pour l'utilisateur lorsque que son inscription est activée et que l'événement commence bientôt.
+        type: LongString
 
-            Cet événement commence bientôt ; si vous ne pouvez plus participer, merci de vous désincrire 
+    ACTIVATED_REGISTRATION_UPCOMING_EVENT_WARNING:
+        content: |
+            Attention : cet événement commence bientôt ! Si vous ne pouvez plus participer, merci de vous désincrire 
             dans les prochaines {grace_period} heures afin de ne pas perturber l'organisation et que
             votre désinscription ne soit pas considérée comme tardive.
-
-            Merci de vérifier si la collective nécessite un paiement de votre part:
-            {link}
-
-            Vous recevez cet e-mail en tant qu'adhérent inscrit à cette collective.
-            Cet e-mail est envoyé par un automate, répondre à cet e-mail sera sans effet.
         description: Template d'email pour l'utilisateur lorsque que son inscription est activée et que l'événement commence bientôt.
         type: LongString
 

collectives/email_templates.py

@@ -14,6 +14,8 @@
     Registration,
     ConfirmationToken,
     BadgeIds,
+    Event,
+    User,
 )
 from collectives.models.auth import ConfirmationTokenType, TokenEmailStatus
 from collectives.utils import mail
@@ -51,19 +53,24 @@ def send_new_event_notification(event):
             current_app.logger.error(f"Mailer error: {err}")
 
 
-def send_unregister_notification(event, user):
+def send_unregister_notification(event: Event, user: User, reason: str):
     """Send a notification to leaders when a user unregisters from an event
 
     :param event: Event on which user unregisters.
-    :type event: :py:class:`collectives.modes.event.Event`
     :param user: User who unregisters.
-    :type user: :py:class:`collectives.models.user.User`
+    :param reason: Use-provided reason for unregistering
     """
     try:
         leader_emails = [l.mail for l in event.leaders]
-        message = Configuration.SELF_UNREGISTER_MESSAGE.format(
+        reason = (
+            f"Justification fournie pour la désinscription : \n {reason}"
+            if reason
+            else ""
+        )
+        message = Configuration.SELF_UNREGISTER_MESSAGE_v2.format(
             user_name=user.full_name(),
             event_title=event.title,
+            reason=reason,
             link=url_for(
                 "event.view_event",
                 event_id=event.id,
@@ -139,18 +146,16 @@ def has_succeed():
         db.session.add(token_copy)
         db.session.commit()
 
-    # Check if local dev, so that email is not sent
-    # and token validation link is displayed in flash popup
-    config = current_app.config
-    if not config["EXTRANET_DISABLE"]:
-        mail.send_mail(
-            email=email,
-            subject=f"{reason.capitalize()} de compte Collectives",
-            message=message,
-            error_action=has_failed,
-            success_action=has_succeed,
-        )
-    else:
+    mail.send_mail(
+        email=email,
+        subject=f"{reason.capitalize()} de compte Collectives",
+        message=message,
+        error_action=has_failed,
+        success_action=has_succeed,
+    )
+
+    # Check if local dev, then token validation link is displayed in flash popup
+    if current_app.config["EXTRANET_DISABLE"]:
         has_succeed()
         url = url_for(
             "auth.process_confirmation", token_uuid=token.uuid, _external=True
@@ -161,19 +166,23 @@ def has_succeed():
         flash(Markup(line))
 
 
-def send_reject_subscription_notification(rejector_name, event, rejected_user_email):
+def send_reject_subscription_notification(
+    rejector_name: str, event: Event, rejected_user_email: str, reason: str
+):
     """Send a notification to user whom registration has been rejected
 
-    :param string rejector_name: User name who rejects the subscription.
+    :param rejector_name: User name who rejects the subscription.
     :param event: Event the registraton is rejected on.
-    :type event: :py:class:`collectives.modes.event.Event`
-    :param string rejected_user_email: User email for who registraton is rejected.
+    :param rejected_user_email: User email for who registraton is rejected.
+    :param reason: Reason for the rejection
     """
     try:
-        message = Configuration.REJECTED_REGISTRATION_MESSAGE.format(
+        reason = f"Justification du refus : \n {reason}" if reason else ""
+        message = Configuration.REJECTED_REGISTRATION_MESSAGE_v2.format(
             rejector_name=rejector_name,
             event_title=event.title,
             event_date=format_date(event.start),
+            reason=reason,
             link=url_for(
                 "event.view_event",
                 event_id=event.id,
@@ -242,32 +251,30 @@ def send_update_waiting_list_notification(
         end_of_grace = current_time() + timedelta(
             hours=Configuration.UNREGISTRATION_GRACE_PERIOD + 1
         )
+
         if registration.is_in_late_unregistration_period(end_of_grace):
-            message = (
-                Configuration.ACTIVATED_REGISTRATION_UPCOMING_EVENT_MESSAGE.format(
-                    event_title=registration.event.title,
-                    event_date=format_date(registration.event.start),
+            unregistration_text = (
+                Configuration.ACTIVATED_REGISTRATION_UPCOMING_EVENT_WARNING.format(
                     grace_period=Configuration.UNREGISTRATION_GRACE_PERIOD,
-                    link=url_for(
-                        "event.view_event",
-                        event_id=registration.event.id,
-                        name=slugify(registration.event.title),
-                        _external=True,
-                    ),
                 )
             )
         else:
-            message = Configuration.ACTIVATED_REGISTRATION_MESSAGE.format(
-                event_title=registration.event.title,
-                event_date=format_date(registration.event.start),
-                link=url_for(
-                    "event.view_event",
-                    event_id=registration.event.id,
-                    name=slugify(registration.event.title),
-                    _external=True,
-                ),
+            unregistration_text = (
+                Configuration.ACTIVATED_REGISTRATION_STANDARD_EVENT_MESSAGE
             )
 
+        message = Configuration.ACTIVATED_REGISTRATION_MESSAGE_v2.format(
+            event_title=registration.event.title,
+            event_date=format_date(registration.event.start),
+            unregistration_text=unregistration_text,
+            link=url_for(
+                "event.view_event",
+                event_id=registration.event.id,
+                name=slugify(registration.event.title),
+                _external=True,
+            ),
+        )
+
         if deleted_registrations:
             event_titles = "\n".join(
                 f" - {reg.event.title}" for reg in deleted_registrations

collectives/routes/event.py

@@ -958,7 +958,8 @@ def self_unregister(event_id):
 
     # Send notification e-mail to leaders only if definitive subscription
     if was_holding_slot:
-        send_unregister_notification(event, current_user)
+        reason = request.form.get("reason")
+        send_unregister_notification(event, current_user, reason)
 
     return redirect(url_for("event.view_event", event_id=event_id))
 
@@ -1160,6 +1161,11 @@ def update_attendance(event_id):
                 RegistrationStatus.Waiting,
                 RegistrationStatus.Rejected,
             ):
+                if previous_status == RegistrationStatus.Waiting:
+                    send_update_waiting_list_notification(
+                        registration, deleted_registrations=[]
+                    )
+
                 # Reset registration time to restart grace period
                 registration.registration_time = current_time()
 
@@ -1171,6 +1177,7 @@ def update_attendance(event_id):
                     current_user.full_name(),
                     registration.event,
                     registration.user.mail,
+                    reason=request.form.get("rejection-reason"),
                 )
 
             if Configuration.ENABLE_SANCTIONS and event.event_type.requires_activity:

collectives/static/css/legacy/event/event-details.scss

@@ -72,6 +72,8 @@
 #attendancelistform td select{
     width: 100%;
 }
+#attendancelistform textarea{
+}
 
 .select {
     height: 40px;

collectives/static/js/event/event.js

@@ -34,3 +34,17 @@ function attendanceSelectAll(value){
                                                 }
                                             });
 }
+
+function attendanceUpdate(select, origValue){
+    if(select.value != origValue) {
+        select.style.color='#eb691c'; 
+        select.style.fontWeight='bold';
+    } else {
+        select.style.color=null; 
+        select.style.fontWeight=null;
+    }
+
+    if(select.value == EnumRegistrationStatusKeys["Rejected"]) {
+        document.getElementById('attendance-rejection-explanation').style.display = null;
+    }
+}

collectives/templates/event/partials/admin.html

@@ -135,7 +135,7 @@ <h5 class="heading-5 collective-display--administration-title"> Inscrire un adh
     </form>
 
     <h5 class="heading-5 collective-display--administration-title"> Liste des présences</h5>
-        <form  action="{{url_for('event.update_attendance', event_id=event.id)}}" method="post" id="attendancelistform" autocomplete="off">
+        <form  class="form" action="{{url_for('event.update_attendance', event_id=event.id)}}" method="post" id="attendancelistform" autocomplete="off">
         <table id="attendancelist">
             <tbody>
                 {% set ns = namespace(license_to_renew=False) %}
@@ -151,7 +151,7 @@ <h5 class="heading-5 collective-display--administration-title"> Liste des prése
                         </a>
                     </td>
                     <td style="text-align:center">
-                        <select name="reg_{{ registration.id }}" onchange="this.style.color='#eb691c'; this.style.fontWeight='bold';">
+                        <select name="reg_{{ registration.id }}" onchange="attendanceUpdate(this, {{registration.status.value}});">
                             {% for status in registration.valid_transitions() %}
                                 <option
                                         value="{{status.value}}"
@@ -183,6 +183,10 @@ <h5 class="heading-5 collective-display--administration-title"> Liste des prése
             = Licence à renouveler <br/>
             {% endif %}
 
+        <textarea id="attendance-rejection-explanation" name="rejection-reason" class="input-50" style="min-width: 22em;" rows="5"
+            placeholder="Explication optionelle pour les personnes nouvellement refusées..."
+        ></textarea>
+
         <input type="hidden" name="csrf_token" value="{{ csrf_token() }}"/><br/>
         <input type="submit" value="Sauvegarder" class="button button-primary">
     </form>

collectives/templates/partials/event/self_unregister.html

@@ -3,7 +3,7 @@
     <form
             action="{{url_for('event.self_unregister', event_id=event.id)}}"
             method="post"
-
+            class="form"
         >
         <input type="hidden" name="csrf_token" value="{{ csrf_token() }}"/>
 
@@ -36,6 +36,9 @@ <h4 class="centeralign">Confirmez-vous votre désinscription ?<br/>
                     <input type="submit" id="submitSelfUnregister" value="Confirmer" class="button button-danger">
                     <input type="button" id="cancelSelfUnregister" value="Annuler" class="button button-secondary" onclick="this.closest('.messages').style.display='none'">
                 </h4>
+                <h4 class="centeralign"> 
+                    <textarea name="reason" class="input-75" placeholder="Justification pour les encadrants..." rows="5" style="min-width:20em; max-width:50em;"></textarea>
+                </h4>
             </div>
             <div class="veil" onclick= "this.parentNode.style.display='none'"></div>
         </div>