.

generic/models.py

@@ -248,10 +248,10 @@ class CidUser(models.Model):
         Passcode: {self.passcode}
         ---------
 
-        You will need these to login to the test system on the day.
+        You will need these to access to the test system.
 
         Direct link to login:
-        {settings.REMOTE_URL}/{login_url}?cid={self.cid}&passcode={self.passcode}
+        {settings.REMOTE_URL}{login_url}?cid={self.cid}&passcode={self.passcode}
 
 
 

generic/templates/generic/group_emailed.html

@@ -1,9 +1,15 @@
 {% extends 'generic/base.html' %}
 
 {% block content %}
+    Attempted to send {{users_count}} emails.
 
-    Emails sent as below:
-    {% for u, success, msg in results %}
-        <p>{{u.cid}}: {{u.email}} [success={{success}}] ({{msg}})</p>
+    <h2>Emails sent:</h2>
+    {% for u in sent %}
+        <p>{{u.cid}}: {{u.email}}</p>
+    {% endfor %}
+
+    <h2>Emails not sent:</h2>
+    {% for u, msg in not_sent %}
+        <p>{{u.cid}}: {{u.email}} [{{msg}}]</p>
     {% endfor %}
 {% endblock %}

generic/views.py

@@ -215,7 +215,9 @@ def generic_exam_json_edit(request):
 
 
 class ExamViews(View, LoginRequiredMixin):
-    def __init__(self, exam, question, cid_user_answer, app, question_type, loadJsonAnswer):
+    def __init__(
+        self, exam, question, cid_user_answer, app, question_type, loadJsonAnswer
+    ):
         self.Exam = exam
         self.Question = question
         self.CidUserAnswer = cid_user_answer
@@ -1028,17 +1030,21 @@ def group_view(request):
 def group_email(request, pk):
     group = get_object_or_404(CidUserGroup, pk=pk)
 
-    users = group.ciduser_set.all()
+    users = group.ciduser_set.all(login_email_sent=False)
 
-    results = []
+    sent = []
+    not_sent = []
     for u in users:
         success, msg = u.email_details()
-        results.append((u, success, msg))
+        if success:
+            sent.append(u)
+        else:
+            not_sent.append((u, msg))
 
     return render(
         request,
         "generic/group_emailed.html",
-        {"results": results},
+        {"sent": sent, "not_sent": not_sent, "users_count": users.count()},
     )