feat(applications): Add field to remove application group from the ap…

…plication page (#765) * add field for showing application group and filter in view * fix wrong mail bug * change names of lists
hackerspace-ntnu · Sep 18, 2023 · 46af547 · 46af547
1 parent 641579d
commit 46af547
Show file tree

Hide file tree

Showing 7 changed files with 61 additions and 44 deletions.
diff --git a/applications/migrations/0019_applicationgroup_open_for_applications.py b/applications/migrations/0019_applicationgroup_open_for_applications.py
@@ -0,0 +1,18 @@
+# Generated by Django 3.2.18 on 2023-09-16 22:28
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('applications', '0018_alter_application_project_interests'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='applicationgroup',
+            name='open_for_applications',
+            field=models.BooleanField(default=True, verbose_name='Åpen for søknader'),
+        ),
+    ]
diff --git a/applications/models.py b/applications/models.py
@@ -43,6 +43,9 @@ class ApplicationGroup(models.Model):
     project_group = models.BooleanField(
         verbose_name="Gruppen tilhører prosjektgruppen", default=False
     )
+    open_for_applications = models.BooleanField(
+        verbose_name="Åpen for søknader", default=True
+    )
 
     def __str__(self):
         return self.name

diff --git a/applications/templates/applications/application_form.html b/applications/templates/applications/application_form.html
@@ -115,7 +115,7 @@ <h5>Velkommen til søknadssiden for Hackerspace NTNU sitt opptak!</h5>
                         {% if form.project_interests.errors %}
                             <span class="helper-text hs-red-text">{{ form.project_interests.errors }}</span>
                         {% else %}
-                            <span class="helper-text">Beskriv gjerne om du ønsker å danne nye, og/eller fortsette på eksisterende prosjekter. Vi ønsker å vite hva som interesserer deg.</span>
+                            <span class="helper-text">Beskriv gjerne om du ønsker å danne nye prosjekter. Vi ønsker å vite hva som interesserer deg.</span>
                         {% endif %}
                     </div>
 

diff --git a/applications/templates/applications/application_info.html b/applications/templates/applications/application_info.html
@@ -55,48 +55,44 @@ <h4>{% trans "Hovedgrupper" %}</h4>
                     </div>
                 </div>
             </div>
-            {% if main_list %}
-                {% for group in main_list %}
-                    {% include "applications/_group_description.html" %}
-                {% empty %}
-                    <div class="section white">
-                        <div class="container">
-                            <div class="row">
-                                <div class="col s12">
-                                    <p>{% trans "Ingen grupper lagt til i opptaket." %}</p>
-                                </div>
-                            </div>
-                        </div>
-                    </div>
-                {% endfor %}
-            {% endif %}
-            {% if group_list %}
-                <div class="divider"></div>
+            {% for group in main_groups %}
+                {% include "applications/_group_description.html" %}
+            {% empty %}
                 <div class="section white">
                     <div class="container">
                         <div class="row">
                             <div class="col s12">
-                                <h4>{% trans "Prosjektgruppene" %}</h4>
-                                <p>{% trans "Prosjektgruppene gjennomfører en rekke prosjekter som varierer fra år til år. Prosjektene varer som oftest i to semestre." %}</p>
-                                <p>{% trans "Som medlem av en prosjektgruppe jobber du konkret på et prosjekt sammen med andre på gruppen." %}</p>
-                                <p>{% trans "Om du har forslag til noen andre prosjekter, kan dette sendes med et søknadsskjema." %}</p>
-                                <p>{% trans "Under finner du informasjon om prosjektene som har opptak nå." %}</p>
+                                <p>{% trans "Ingen grupper lagt til i opptaket." %}</p>
                             </div>
                         </div>
                     </div>
                 </div>
-                {% for group in group_list %}
-                    {% include "applications/_group_description.html" %}
-                {% empty %}
-                    <div class="section white">
-                        <div class="container">
-                            <div class="row">
-                                <div class="col s12">
-                                    <p>{% trans "Ingen grupper lagt til i opptaket." %}</p>
-                                </div>
+            {% endfor %}
+            <div class="divider"></div>
+            <div class="section white">
+                <div class="container">
+                    <div class="row">
+                        <div class="col s12">
+                            <h4>{% trans "Prosjektgruppene" %}</h4>
+                            <p>{% trans "Prosjektgruppene gjennomfører en rekke prosjekter som varierer fra år til år. Prosjektene varer som oftest i to semestre." %}</p>
+                            <p>{% trans "Som medlem av en prosjektgruppe jobber du konkret på et prosjekt sammen med andre på gruppen." %}</p>
+                            <p>{% trans "Om du har forslag til noen andre prosjekter, kan dette sendes med et søknadsskjema." %}</p>
+                            <p>{% trans "Under finner du informasjon om prosjektene som har opptak nå." %}</p>
+                        </div>
+                    </div>
+                </div>
+            </div>
+            {% for group in project_groups %}
+                {% include "applications/_group_description.html" %}
+            {% empty %}
+                <div class="section white">
+                    <div class="container">
+                        <div class="row">
+                            <div class="col s12">
+                                <p>{% trans "Ingen grupper lagt til i opptaket." %}</p>
                             </div>
                         </div>
                     </div>
-                {% endfor %}
-            {% endif %}
+                </div>
+            {% endfor %}
 {% endblock %}
diff --git a/applications/tests.py b/applications/tests.py
@@ -47,8 +47,8 @@ def setUp(self):
         self.response = self.client.get(reverse("application:application_info"))
 
     def test_context(self):
-        self.assertIsNotNone(self.response.context["group_list"])
-        self.assertIsNotNone(self.response.context["main_list"])
+        self.assertIsNotNone(self.response.context["project_groups"])
+        self.assertIsNotNone(self.response.context["main_groups"])
 
 
 class ApplicationFormViewTest(TestCase):

diff --git a/applications/views.py b/applications/views.py
@@ -3,22 +3,22 @@
 
 from applications.forms import ApplicationForm
 
-from .models import ApplicationGroup, ApplicationPeriod
+from .models import ApplicationGroup
 
 
 class ApplicationInfoView(ListView):
     template_name = "applications/application_info.html"
     queryset = ApplicationGroup.objects.all()
 
     def get_context_data(self, **kwargs):
-        period = ApplicationPeriod.objects.filter(name="Opptak").first()
         return {
             **super().get_context_data(**kwargs),
-            "group_list": ApplicationGroup.objects.filter(project_group=True),
-            "main_list": ApplicationGroup.objects.filter(project_group=False),
-            "start_date": period.period_start if period else None,
-            "end_date": period.period_end if period else None,
-            "period_status": period.status() if period else None,
+            "project_groups": ApplicationGroup.objects.filter(
+                project_group=True, open_for_applications=True
+            ),
+            "main_groups": ApplicationGroup.objects.filter(
+                project_group=False, open_for_applications=True
+            ),
         }
 
 

diff --git a/website/templates/website/footer.html b/website/templates/website/footer.html
@@ -21,7 +21,7 @@ <h5>Hackerspace</h5>
                         {% trans "for detaljerte åpningstider" %}
                     </p>
                 </div>
-                <p>{% trans "For øvrige henvendelser kan du også ta kontakt via" %} <a class="white-text" style="white-space: nowrap;" href="mailto:herspace[email protected]">[email protected]</a>
+                <p>{% trans "For øvrige henvendelser kan du også ta kontakt via" %} <a class="white-text" style="white-space: nowrap;" href="mailto:hackerspace[email protected]">[email protected]</a>
                 </div>
                 <div class="col m6 l4 hide-on-small-only">
                     <h5>Lenker</h5>