From 0347adc2676e08cecb29e9b9c6ba55b2ec477c6f Mon Sep 17 00:00:00 2001
From: Luca <Luca@hackerspace-bamberg.de>
Date: Sun, 14 May 2023 14:27:13 +0200
Subject: [PATCH] Show checkin state for running shifts/shifts in the next 30
 minutes

---
 .../team/templates/partials/shift_box.html    |  8 +++-
 .../team/templates/shift_overview.html        | 12 ++++-
 shiftregister/team/urls.py                    |  2 +-
 shiftregister/team/views.py                   | 48 +++++++++++++++----
 4 files changed, 56 insertions(+), 14 deletions(-)

diff --git a/shiftregister/team/templates/partials/shift_box.html b/shiftregister/team/templates/partials/shift_box.html
index 822b034..189461b 100644
--- a/shiftregister/team/templates/partials/shift_box.html
+++ b/shiftregister/team/templates/partials/shift_box.html
@@ -7,8 +7,12 @@
         <strong>Beginn:</strong> {{ shift.start_at }}<br>
         <strong>Dauer:</strong> {{ shift.duration }}<br>
         <strong>Belegung:</strong> {{ shift.registration_count }}/{{ shift.required_helpers|default:shift.room.required_helpers }}
+{% if shift.checkin_count is not None %}
+        <br>
+        <strong>Checkin-Status:</strong> {% if shift.checkin_count == shift.required_helpers|default:shift.room.required_helpers %}<span class="tag is-rounded is-success">vollständig</span>{% elif shift.checkin_count > 0 %}<span class="tag is-rounded is-warning">teilweise</span>{% else %}<span class="tag is-rounded is-danger">kein Checkin</span>{% endif %}
+{% endif %}
     </div>
-    <div class="is-flex is-justify-content-end">
-        <a class="button is-info is-small" href="{% url 'team:shift' shift.id %}">Details</a>
+    <div class="buttons is-right mt-3">
+        <a class="button is-info is-small mr-0" href="{% url 'team:shift' shift.id %}">Details</a>
     </div>
 </div>
diff --git a/shiftregister/team/templates/shift_overview.html b/shiftregister/team/templates/shift_overview.html
index ddd7aae..0f4aa6c 100644
--- a/shiftregister/team/templates/shift_overview.html
+++ b/shiftregister/team/templates/shift_overview.html
@@ -12,12 +12,20 @@
 </div>
 <hr>
 {% endif %}
-<h3 class="title is-spaced">Nächste Schichten pro Raum</h3>
+{% if next_shifts %}
+<h3 class="title is-spaced">Nächste Schichten</h3>
 <div class="columns is-multiline">
 {% for shift in next_shifts %}
+    {% include "partials/shift_list_item.html" %}
+{% endfor %}
+</div>
+{% endif %}
+<h3 class="title is-spaced">Nächste Schichten pro Raum</h3>
+<div class="columns is-multiline">
+{% for shift in next_shifts_per_room %}
     <div class="column is-one-quarter">
         <h5 class="subtitle"><a href="{% url 'team:shift_room' shift.room.name %}">{{ shift.room.name }}</a></h5>
-        <a class="" href="{% url 'team:shift' shift.id %}">
+        <a href="{% url 'team:shift' shift.id %}">
             {% include "partials/shift_box.html" %}
         </a>
     </div>
diff --git a/shiftregister/team/urls.py b/shiftregister/team/urls.py
index f8a4296..c76bf56 100644
--- a/shiftregister/team/urls.py
+++ b/shiftregister/team/urls.py
@@ -4,7 +4,7 @@ from . import views
 
 app_name = "team"
 urlpatterns = [
-    path("", views.shift_overview, name="index"),
+    path("", views.index, name="index"),
     path("overview/", views.shift_overview, name="shift_overview"),
     path("shifts/", views.ShiftList.as_view(), name="shift_all"),
     path("room_shifts/<path:pk>", views.RoomShiftList.as_view(), name="shift_room"),
diff --git a/shiftregister/team/views.py b/shiftregister/team/views.py
index 7a26a05..f119157 100644
--- a/shiftregister/team/views.py
+++ b/shiftregister/team/views.py
@@ -11,31 +11,61 @@ from django.contrib import messages
 from django.db import models
 from django.core.paginator import Paginator
 from .forms import BulkMessage, HelperShift, HelperMessage
+from datetime import timedelta
 
 
 # Create your views here.
+
+
 def index(request):
-    pass
+    return redirect("team:shift_overview")
 
 
 @login_required
 def shift_overview(request):
+    checkin_count = Count(
+        Case(
+            When(
+                shiftregistration__state=ShiftRegistration.RegState.CHECKED_IN, then=1
+            ),
+            output_field=models.IntegerField(),
+        )
+    )
+
     context = {}
-    context["running_shifts"] = [
-        shift
-        for shift in Shift.with_reg_count()
+    context["running_shifts"] = (
+        Shift.with_reg_count()
         .annotate(
+            checkin_count=Count(
+                Case(
+                    When(
+                        shiftregistration__state=ShiftRegistration.RegState.CHECKED_IN,
+                        then=1,
+                    ),
+                    output_field=models.IntegerField(),
+                )
+            ),
             end_at=ExpressionWrapper(
-                F("start_at") + F("duration"),
-                output_field=DateTimeField(),
-            )
+                F("start_at") + F("duration"), output_field=DateTimeField()
+            ),
         )
         .filter(start_at__lte=timezone.now(), end_at__gte=timezone.now(), deleted=False)
         .order_by("start_at")
-    ]
+    )
+
+    context["next_shifts"] = (
+        Shift.with_reg_count()
+        .annotate(checkin_count=checkin_count)
+        .filter(
+            start_at__gt=timezone.now(),
+            start_at__lte=timezone.now() + timedelta(minutes=30),
+            deleted=False,
+        )
+        .order_by("start_at")
+    )
 
     # only Postgres supports DISTINCT on specific columns, SQLite does not support aggregates on datetime fields
-    context["next_shifts"] = filter(
+    context["next_shifts_per_room"] = filter(
         lambda x: x is not None,
         (
             Shift.with_reg_count()