Merge branch 'master' into 702-fix-login-page

backend/root/management/commands/seed_cypress.py

@@ -58,6 +58,7 @@ def handle(self, *args, **options):
 
             Venue.objects.create(
                 name='Cypress Area',
+                slug='cypress-area',
                 description='Some description',
                 floor=1,
                 last_renovated=timezone.now(),

backend/root/management/commands/seed_scripts/information_pages.py

@@ -1,3 +1,5 @@
+from django.utils.text import slugify
+
 from root.utils.samfundet_random import words
 
 from samfundet.models.general import InformationPage
@@ -14,7 +16,7 @@ def seed():
     for i in range(COUNT):
         # Event title and time
         title_nb, title_en = words(2, include_english=True)
-        slug_field = title_nb.lower().replace(' ', '-')
+        slug_field = slugify(title_nb)
 
         # Make sure slug field is unique
         if slug_field in used_slug_fields:

backend/root/management/commands/seed_scripts/recruitment.py

@@ -41,23 +41,27 @@ def seed():
 
     total_recruitments = len(recruitments) * len(organizations)
     created_recruitments = 0
+    recruitment_objects = []
 
     for org_name in organizations:
         org = Organization.objects.get(name=org_name)
         for recruitment_data in recruitments:
-            Recruitment.objects.get_or_create(
+            recruitment_instance = Recruitment(
                 name_nb=recruitment_data['name_nb'],
                 name_en=recruitment_data['name_en'],
                 organization=org,
-                defaults={
-                    'visible_from': recruitment_data['visible_from'],
-                    'shown_application_deadline': recruitment_data['shown_application_deadline'],
-                    'actual_application_deadline': recruitment_data['actual_application_deadline'],
-                    'reprioritization_deadline_for_applicant': recruitment_data['reprioritization_deadline_for_applicant'],
-                    'reprioritization_deadline_for_groups': recruitment_data['reprioritization_deadline_for_groups'],
-                }
+                visible_from=recruitment_data['visible_from'],
+                shown_application_deadline=recruitment_data['shown_application_deadline'],
+                actual_application_deadline=recruitment_data['actual_application_deadline'],
+                reprioritization_deadline_for_applicant=recruitment_data['reprioritization_deadline_for_applicant'],
+                reprioritization_deadline_for_groups=recruitment_data['reprioritization_deadline_for_groups']
             )
+            recruitment_objects.append(recruitment_instance)
+
             created_recruitments += 1
             yield (created_recruitments / total_recruitments) * 100, 'recruitment'
 
+    # Using bulk_create to add all recruitment instances to the database
+    Recruitment.objects.bulk_create(recruitment_objects)
+
     yield 100, f'Created {created_recruitments} recruitments'

backend/root/management/commands/seed_scripts/recruitment_admissions.py

@@ -7,10 +7,8 @@
 ADMISSION_DATA = {
     'admission_text': 'This is the admission text',
     'applicant_priority': 0,
-    'interview_time': None,
-    'interview_location': None,
     'recruiter_priority': 0,
-    'recruiter_status': 0
+    'recruiter_status': 0,
 }
 
 

backend/root/utils/mixins.py

@@ -162,3 +162,14 @@ def from_db(cls, db, field_names, values):  # type: ignore # noqa: ANN001,ANN206
         setattr(instance, instance._FTM_LOADED_FIELDS_NAME, loaded_fields)  # noqa: FKA01
 
         return instance
+
+
+class FullCleanSaveMixin(Model):
+    """Mixin to call full_clean() before save()."""
+
+    class Meta:
+        abstract = True
+
+    def save(self, *args: Any, **kwargs: Any) -> None:
+        self.full_clean()
+        super().save(*args, **kwargs)

backend/samfundet/admin.py

@@ -14,7 +14,13 @@
     CustomGuardedModelAdmin,
 )
 from .models.event import (Event, EventGroup, EventRegistration)
-from .models.recruitment import (Recruitment, RecruitmentPosition, RecruitmentAdmission, InterviewRoom)
+from .models.recruitment import (
+    Recruitment,
+    RecruitmentPosition,
+    RecruitmentAdmission,
+    InterviewRoom,
+    Interview,
+)
 from .models.general import (
     Tag,
     User,
@@ -536,24 +542,18 @@ class RecruitmentAdmissionAdmin(CustomGuardedModelAdmin):
         'id',
         'recruitment_position',
         'recruitment',
-        'interview_time',
-        'interview_location',
         'user',
     ]
     list_display = [
         'id',
         'recruitment_position',
         'recruitment',
-        'interview_time',
-        'interview_location',
         'user',
     ]
     search_fields = [
         'id',
         'recruitment_position',
         'recruitment',
-        'interview_time',
-        'interview_location',
         'user',
     ]
     list_select_related = True
@@ -576,4 +576,12 @@ class InterviewRoomAdmin(CustomGuardedModelAdmin):
     list_select_related = ['recruitment', 'gang']
 
 
+@admin.register(Interview)
+class InterviewAdmin(CustomGuardedModelAdmin):
+    list_filter = ['id', 'notes']
+    list_display = ['id', 'notes']
+    search_fields = ['id', 'notes']
+    list_display_links = ['id', 'notes']
+
+
 ### End: Our models ###

backend/samfundet/conftest.py

@@ -13,7 +13,7 @@
 from samfundet.models.billig import BilligEvent
 from samfundet.models.event import Event, EventAgeRestriction, EventTicketType
 from samfundet.models.recruitment import Recruitment, RecruitmentPosition, RecruitmentAdmission
-from samfundet.models.general import User, Image, InformationPage, Organization, Gang, BlogPost
+from samfundet.models.general import User, Image, InformationPage, Organization, Gang, BlogPost, TextItem
 
 import root.management.commands.seed_scripts.billig as billig_seed
 """
@@ -193,6 +193,17 @@ def fixture_gang(fixture_organization: Organization) -> Iterator[Gang]:
     organization.delete()
 
 
+@pytest.fixture
+def fixture_text_item() -> Iterator[TextItem]:
+    text_item = TextItem.objects.create(
+        key='foo',
+        text_nb='foo',
+        text_en='foo',
+    )
+    yield text_item
+    text_item.delete()
+
+
 @pytest.fixture
 def fixture_recruitment(fixture_organization: Organization) -> Iterator[Recruitment]:
     now = timezone.now()

backend/samfundet/migrations/0039_remove_recruitmentadmission_interview_location_and_more.py

@@ -0,0 +1,52 @@
+# Generated by Django 4.2.3 on 2023-10-07 22:02
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('samfundet', '0038_alter_recruitmentadmission_interview_location_and_more'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='recruitmentadmission',
+            name='interview_location',
+        ),
+        migrations.RemoveField(
+            model_name='recruitmentadmission',
+            name='interview_time',
+        ),
+        migrations.RemoveField(
+            model_name='recruitmentadmission',
+            name='room',
+        ),
+        migrations.AddField(
+            model_name='recruitmentposition',
+            name='shared_interview_positions',
+            field=models.ManyToManyField(blank=True, help_text='Positions with shared interview', to='samfundet.recruitmentposition'),
+        ),
+        migrations.AlterField(
+            model_name='recruitmentposition',
+            name='interviewers',
+            field=models.ManyToManyField(blank=True, help_text='Interviewers for the position', related_name='interviewers', to=settings.AUTH_USER_MODEL),
+        ),
+        migrations.CreateModel(
+            name='Interview',
+            fields=[
+                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('interview_time', models.DateTimeField(blank=True, help_text='The time of the interview', null=True)),
+                ('interview_location', models.CharField(blank=True, help_text='The location of the interview', max_length=255, null=True)),
+                ('notes', models.TextField(blank=True, help_text='Notes for the interview', null=True)),
+                ('room', models.ForeignKey(blank=True, help_text='Room where the interview is held', null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='interviews', to='samfundet.interviewroom')),
+            ],
+        ),
+        migrations.AddField(
+            model_name='recruitmentadmission',
+            name='interview',
+            field=models.ForeignKey(blank=True, help_text='The interview for the admission', null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='admissions', to='samfundet.interview'),
+        ),
+    ]