From 0b58d47dbbe84ec48f3d85ff8dc7be06c0b68c4b Mon Sep 17 00:00:00 2001
From: Luca <Luca@hackerspace-bamberg.de>
Date: Tue, 17 May 2022 17:28:17 +0200
Subject: [PATCH] Style AStA membership confirmation

---
 shiftregister/app/forms.py                   |  4 ++--
 shiftregister/app/templates/asta.html        |  4 ++--
 shiftregister/app/templates/helper_base.html | 14 +++-----------
 shiftregister/app/views.py                   |  4 ++--
 4 files changed, 9 insertions(+), 17 deletions(-)

diff --git a/shiftregister/app/forms.py b/shiftregister/app/forms.py
index 7f81b0c..8bc463b 100644
--- a/shiftregister/app/forms.py
+++ b/shiftregister/app/forms.py
@@ -42,8 +42,8 @@ class RegisterForm(forms.Form):
 class AstaForm(forms.Form):
     confirm = forms.BooleanField(
         required=True,
-        label="Ich habe mich beim AStA als reguläres Mitglied angemeldet",
-        help_text="Aus Versicherungsgründen müssen Helfer*innen AStA Mitglieder sein. Die Mitgliedschaft ist kostenlos.",
+        label="Ich habe mich beim AStA als reguläres Mitglied angemeldet.",
+        help_text="Aus Versicherungsgründen müssen Helfer*innen AStA-Mitglieder sein. Die Mitgliedschaft ist kostenlos.",
     )
 
 
diff --git a/shiftregister/app/templates/asta.html b/shiftregister/app/templates/asta.html
index 950e86a..27de74b 100644
--- a/shiftregister/app/templates/asta.html
+++ b/shiftregister/app/templates/asta.html
@@ -1,9 +1,9 @@
 {% extends "helper_base.html" %}
 
-{% block title %}Registrierung{% endblock %}
+{% block title %}AStA-Mitgliedschaft{% endblock %}
 
 {% block content %}
-<a href="https://asta-bamberg.de/mitglied-werden" target="_blank" class="button is-primary">AStA Formular</a>
+<a class="button is-primary mb-2" href="https://asta-bamberg.de/mitglied-werden" target="_blank">AStA-Formular</a>
 <form action="" method="post">
     {% csrf_token %}
 {% for field in form %}
diff --git a/shiftregister/app/templates/helper_base.html b/shiftregister/app/templates/helper_base.html
index a089c9c..6a02fb5 100644
--- a/shiftregister/app/templates/helper_base.html
+++ b/shiftregister/app/templates/helper_base.html
@@ -2,10 +2,6 @@
 
 {% block title %}Help!{% endblock %}
 
-{% block custom_notifications %}
-
-{% endblock %}
-
 {% block navbar %}
 <div class="navbar-start">
 {% if not helper%}
@@ -23,17 +19,13 @@
 {% endblock %}
 
 {% block body %}
+<section class="section">
+    <div class="container">
 {% if helper and not helper.asta_confirmed %}
-<section class="section">
-    <div class="container">
         <div class="notification is-danger">
-            <a href="{% url 'asta' %}">Bitte Bestätige deine AStA Mitgliedschaft</a>
+            <a href="{% url 'asta' %}">Bitte bestätige deine AStA-Mitgliedschaft</a>
         </div>
-    </div>
-</section>
 {% endif %}
-<section class="section">
-    <div class="container">
 {% include 'notifications.html' %}
 {% if helper %}
         <h1 class="title is-1">Hallo {{ helper.name }}!</h1>
diff --git a/shiftregister/app/views.py b/shiftregister/app/views.py
index ce23c19..13b18f9 100644
--- a/shiftregister/app/views.py
+++ b/shiftregister/app/views.py
@@ -252,7 +252,7 @@ def cancel(request, shiftid):
 
 
 def asta(request):
-    if not request.helper:
+    if not request.helper or request.helper.asta_confirmed:
         return redirect("index")
     form = AstaForm()
     if request.method == "POST":
@@ -263,7 +263,7 @@ def asta(request):
             messages.add_message(
                 request,
                 messages.SUCCESS,
-                "Deine AStA-Mitgliedschaft wurde gespeichert",
+                "Deine AStA-Mitgliedschaft wurde gespeichert.",
             )
             return redirect("index")