From 7c1276fa19c16da42fd8a4943aee928229867b6c Mon Sep 17 00:00:00 2001
From: Alexandre Paice <alexandre.paice@ugent.be>
Date: Mon, 20 May 2024 15:48:56 +0200
Subject: [PATCH 01/25] unused import

---
 .../src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
index 070550be..857cc83f 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
@@ -1,4 +1,4 @@
-import React, { useState } from 'react'
+import { useState } from 'react'
 import {
     Box,
     TextField,

From f3f6387647ffafc08199cf24aaa0b09cf3904123 Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 16:11:09 +0200
Subject: [PATCH 02/25] logo login terug gezet

---
 frontend/frontend/src/pages/loginPage/LoginPage.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/frontend/src/pages/loginPage/LoginPage.tsx b/frontend/frontend/src/pages/loginPage/LoginPage.tsx
index ff1e209a..28b4cb29 100644
--- a/frontend/frontend/src/pages/loginPage/LoginPage.tsx
+++ b/frontend/frontend/src/pages/loginPage/LoginPage.tsx
@@ -81,7 +81,7 @@ export function LoginPage() {
                         alignSelf={'center'}
                     >
                         <Box
-                            id='logo'
+                            id="logo"
                             component="img"
                             src={t('logo_blue')}
                             alt="logo"
@@ -93,7 +93,7 @@ export function LoginPage() {
                         />
                         <Box
                             component="img"
-                            src={'assets/logo_duif.png'}
+                            src={'/assets/logo_duif.png'}
                             alt="logo_app"
                             sx={{
                                 padding: 0,

From 414d4c341cf4ccc8fec03c809962da942f8cc1fe Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 17:21:14 +0200
Subject: [PATCH 03/25] assignmentpage zou in orde moeten zijn terug, lijn per
 lijn nagekeken met oude commit

---
 .../pages/assignmentPage/AssignmentPage.tsx   | 115 ++++++++----------
 1 file changed, 52 insertions(+), 63 deletions(-)

diff --git a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
index a5da5ff8..9a87337e 100644
--- a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
+++ b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
@@ -32,6 +32,7 @@ import WarningPopup from '../../components/WarningPopup.tsx'
 import { User } from '../subjectsPage/AddChangeSubjectPage.tsx'
 import StudentPopUp from '../subjectsPage/StudentPopUp.tsx'
 import axios, { AxiosResponse } from 'axios'
+import { GroupAccessComponent } from '../../components/GroupAccessComponent.tsx'
 
 // group interface
 export interface Group {
@@ -440,60 +441,59 @@ export function AssignmentPage() {
                                         />
                                     ) : (
                                         <>
-                                            <>
-                                                {assignment?.student_groep ? (
-                                                    <Button
-                                                        sx={{
-                                                            bgcolor:
-                                                                'secondary.main',
-                                                            textTransform:
-                                                                'none',
-                                                        }}
-                                                        onClick={goToGroups}
-                                                    >
-                                                        <Typography color="secondary.contrastText">
-                                                            {t('group')}
-                                                        </Typography>
-                                                    </Button>
-                                                ) : (
-                                                    <Box
-                                                        sx={{
-                                                            position:
-                                                                'absolute',
-                                                            top: 90,
-                                                            right: 50,
-                                                            display: 'flex',
-                                                            justifyContent:
-                                                                'flex-end',
-                                                            zIndex: 1,
-                                                            marginTop: '-40px',
-                                                        }}
-                                                    >
-                                                        {assignment?.max_groep_grootte ===
-                                                        1 ? (
-                                                            <Typography variant="body1">
-                                                                {user.first_name +
-                                                                    ' ' +
-                                                                    user.last_name}
-                                                            </Typography>
-                                                        ) : (
-                                                            <>
-                                                                <StudentPopUp
-                                                                    students={
-                                                                        studentsLoading
-                                                                            ? []
-                                                                            : students
-                                                                    }
-                                                                    text="group_members"
-                                                                />
-                                                            </>
-                                                        )}
-                                                    </Box>
-                                                )}
-                                            </>
+                                            {(assignment?.max_groep_grootte
+                                                ? assignment.max_groep_grootte
+                                                : 1) > 1 && (
+                                                <GroupAccessComponent
+                                                    assignmentid={parseInt(
+                                                        assignmentId
+                                                    )}
+                                                    courseid={parseInt(
+                                                        courseId
+                                                    )}
+                                                />
+                                            )}
                                         </>
                                     )}
                                 </Box>
+                                {/*extra deadline*/}
+                                {assignment?.extra_deadline && (
+                                    <Box
+                                        display={'flex'}
+                                        flexDirection={'row'}
+                                        alignItems={'center'}
+                                        gap={1}
+                                        pl={2.5}
+                                    >
+                                        <Typography
+                                            variant="h6"
+                                            color="text.primary"
+                                            fontWeight={'bold'}
+                                        >
+                                            Extra Deadline:
+                                        </Typography>
+                                        {loading ? (
+                                            <Skeleton
+                                                variant="text"
+                                                width={180}
+                                                height={50}
+                                            />
+                                        ) : (
+                                            <Typography
+                                                variant="h6"
+                                                color={'text.primary'}
+                                            >
+                                                {assignment
+                                                    ? dayjs(
+                                                          assignment.extra_deadline
+                                                      ).format(
+                                                          'DD/MM/YYYY-HH:MM'
+                                                      )
+                                                    : 'no deadline'}
+                                            </Typography>
+                                        )}
+                                    </Box>
+                                )}
 
                                 {/* Assignment description */}
                                 <Card sx={{ padding: '20px' }}>
@@ -1057,18 +1057,7 @@ export function AssignmentPage() {
                                                                         )}
                                                                         <SubmissionListItemStudentPage
                                                                             realId={submission.indiening_id.toString()}
-                                                                            visualId={(
-                                                                                submission.indiening_id -
-                                                                                Math.min(
-                                                                                    ...submissions.map(
-                                                                                        (
-                                                                                            submission
-                                                                                        ) =>
-                                                                                            submission.indiening_id
-                                                                                    )
-                                                                                ) +
-                                                                                1
-                                                                            ).toString()}
+                                                                            visualId={index.toString()}
                                                                             timestamp={dayjs(
                                                                                 submission.tijdstip
                                                                             ).format(

From 57d451158727d7a5cdea881eb5e255f12ca7bc01 Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 17:26:49 +0200
Subject: [PATCH 04/25] index laten beginnen van 1 ipv 0

---
 .../frontend/src/pages/assignmentPage/AssignmentPage.tsx     | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
index 9a87337e..5d13f24f 100644
--- a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
+++ b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
@@ -1057,7 +1057,10 @@ export function AssignmentPage() {
                                                                         )}
                                                                         <SubmissionListItemStudentPage
                                                                             realId={submission.indiening_id.toString()}
-                                                                            visualId={index.toString()}
+                                                                            visualId={(
+                                                                                submissions.length -
+                                                                                index
+                                                                            ).toString()}
                                                                             timestamp={dayjs(
                                                                                 submission.tijdstip
                                                                             ).format(

From 20101c0e75e784a3d04566aedbcdc559a7d4c068 Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 18:12:19 +0200
Subject: [PATCH 05/25] tijdsformaat van dayjs gefixt op assignmentpage

---
 .../frontend/src/pages/assignmentPage/AssignmentPage.tsx  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
index 5d13f24f..fe9fb20f 100644
--- a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
+++ b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
@@ -421,7 +421,7 @@ export function AssignmentPage() {
                                                                 ? dayjs(
                                                                       assignment.deadline
                                                                   ).format(
-                                                                      'DD/MM/YYYY-HH:MM'
+                                                                      'DD/MM/YYYY HH:mm'
                                                                   )
                                                                 : 'no deadline'}
                                                         </Typography>
@@ -487,7 +487,7 @@ export function AssignmentPage() {
                                                     ? dayjs(
                                                           assignment.extra_deadline
                                                       ).format(
-                                                          'DD/MM/YYYY-HH:MM'
+                                                          'DD/MM/YYYY HH:mm'
                                                       )
                                                     : 'no deadline'}
                                             </Typography>
@@ -772,7 +772,7 @@ export function AssignmentPage() {
                                                                     ? dayjs(
                                                                           assignment.deadline
                                                                       ).format(
-                                                                          'DD/MM/YYYY-HH:MM'
+                                                                          'DD/MM/YYYY HH:mm'
                                                                       )
                                                                     : 'no deadline'}
                                                             </Typography>
@@ -877,7 +877,7 @@ export function AssignmentPage() {
                                                     ? dayjs(
                                                           assignment.extra_deadline
                                                       ).format(
-                                                          'DD/MM/YYYY-HH:MM'
+                                                          'DD/MM/YYYY HH:mm'
                                                       )
                                                     : 'no deadline'}
                                             </Typography>

From d5aa7ecc77179d96dc8d8e562aa4a80a1df63c3c Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 18:16:05 +0200
Subject: [PATCH 06/25] kleine taalfix GroepsNummer -> Groepnummer

---
 frontend/frontend/src/i18n/nl.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/frontend/src/i18n/nl.ts b/frontend/frontend/src/i18n/nl.ts
index ea6fe94d..cb7a9b09 100644
--- a/frontend/frontend/src/i18n/nl.ts
+++ b/frontend/frontend/src/i18n/nl.ts
@@ -116,7 +116,7 @@ const dutch = {
     noGroup: 'Je lijkt nog geen groep te hebben.',
     chooseGroup: 'Kies een groep voor je indient',
     contactTeacher: 'Gelieve contact op te nemen met je lesgever.',
-    group_number: 'GroepsNummer',
+    group_number: 'Groepnummer',
     open_with_ui: 'Open met UI',
     choose_existing: 'Kies een bestaand script',
     make_new_script: 'Maak nieuw script aan',

From e98c1a70938e7c9e04106d280e0442030239b9a6 Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 18:49:47 +0200
Subject: [PATCH 07/25] 'geen score' op subjectPage ipv 0/20

---
 .../pages/subjectsPage/AssignmentListItemSubjectsPage.tsx   | 4 ++--
 frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx   | 6 ------
 2 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx b/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
index 286bca7c..66c2570a 100644
--- a/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
@@ -33,7 +33,7 @@ interface AssignmentListItemSubjectsPageProps {
     projectName: string
     dueDate: Dayjs
     submissions: number
-    score: Score
+    score: Score | undefined
     maxScore: number
     isStudent: boolean
     archived: boolean
@@ -119,7 +119,7 @@ export function AssignmentListItemSubjectsPage({
                                         />
                                     ) : (
                                         <ListItemText
-                                            primary={`0/${maxScore} (0%)`}
+                                            primary={t('no_score_yet')}
                                         />
                                     )}
                                 </>,
diff --git a/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx b/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
index 0e7a64e4..caa0953d 100644
--- a/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
@@ -282,12 +282,6 @@ export function ProjectsView({
                                                     }
                                                     score={
                                                         project.score
-                                                            ? project.score
-                                                            : {
-                                                                  score_id: 0,
-                                                                  score: 0,
-                                                                  indiening: 0,
-                                                              }
                                                     }
                                                     maxScore={Number(
                                                         project.assignment

From b31e8f17436e9196c7deb0dac565f65ee12aca03 Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 18:58:21 +0200
Subject: [PATCH 08/25] cancel + alert add change subject

---
 .../subjectsPage/AddChangeSubjectPage.tsx     | 22 +++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
index c49aad4c..2d33ca2a 100644
--- a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
@@ -127,7 +127,6 @@ function UserList(
                                             ]}
                                         />
                                     </ListItem>
-
                                 </>
                             )
                         })}
@@ -461,6 +460,7 @@ export function AddChangeSubjectPage() {
                 })
                 .catch((err) => {
                     console.log(err)
+                    alert(err.response.data)
                 })
         }
     }
@@ -564,7 +564,7 @@ export function AddChangeSubjectPage() {
                         <>
                             <Stack direction={'column'}>
                                 <Header
-                                    variant={loading ? 'default' : 'not_main'}
+                                    variant={'default'}
                                     title={loading ? '' : title}
                                 />
                                 <Stack
@@ -622,13 +622,27 @@ export function AddChangeSubjectPage() {
                                                 />
                                             )}
                                         </Box>
-                                        <Box padding={'20px'}>
+                                        <Box
+                                            padding={'20px'}
+                                            display={'flex'}
+                                            flexDirection={'row'}
+                                            gap={2}
+                                        >
                                             <SecondaryButton
+                                                /* This is the large save button on the top of the page */
+                                                onClick={() =>
+                                                    navigate(`/course/${vakID}`)
+                                                }
+                                            >
+                                                {t('cancel')}
+                                            </SecondaryButton>
+
+                                            <Button
                                                 /* This is the large save button on the top of the page */
                                                 onClick={handleSave}
                                             >
                                                 {t('save')}
-                                            </SecondaryButton>
+                                            </Button>
                                         </Box>
                                     </Box>
                                     <Stack direction={'row'} gap={10}>

From 2a9f0d263bf97eaedbb60b1870f13f65b6039789 Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 19:05:05 +0200
Subject: [PATCH 09/25] kotjes deadlines terug opgeschoond

---
 .../src/components/DeadlineCalendar.tsx       | 25 ++++++++++++++-----
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/frontend/frontend/src/components/DeadlineCalendar.tsx b/frontend/frontend/src/components/DeadlineCalendar.tsx
index 51c041a8..f5007ba6 100644
--- a/frontend/frontend/src/components/DeadlineCalendar.tsx
+++ b/frontend/frontend/src/components/DeadlineCalendar.tsx
@@ -7,13 +7,13 @@ import {
 import dayjs, { Dayjs } from 'dayjs'
 import {
     Badge,
-    SxProps,
-    Stack,
-    Typography,
     List,
     ListItem,
     ListItemButton,
     ListItemText,
+    Stack,
+    SxProps,
+    Typography,
 } from '@mui/material'
 import { useEffect, useRef, useState } from 'react'
 import AssignmentIcon from '@mui/icons-material/Assignment'
@@ -125,7 +125,14 @@ function DeadlineMenu({ assignments, selectedDay }: DeadlineMenuProps) {
                         <ListItem>
                             <ListItemButton
                                 sx={{
-                                    border: 1,
+                                    border: 0.5,
+                                    borderColor: 'primary.main',
+                                    textAlign: 'center',
+                                    width: '100%',
+                                    borderRadius: 2,
+                                    '&:hover': {
+                                        backgroundColor: 'primary.light',
+                                    },
                                 }}
                                 onClick={() =>
                                     handleProjectClick(
@@ -134,8 +141,14 @@ function DeadlineMenu({ assignments, selectedDay }: DeadlineMenuProps) {
                                     )
                                 }
                             >
-                                <ListItemText primary={assignment.titel} />
-
+                                <ListItemText sx={{ maxWidth: '100%' }}>
+                                    <Typography
+                                        textOverflow={'ellipsis'}
+                                        color={'text.primary'}
+                                    >
+                                        {assignment.titel}
+                                    </Typography>
+                                </ListItemText>
                             </ListItemButton>
                         </ListItem>
                     ))}

From 3fe2e92890b43da55046c40016b279296f4d2c7e Mon Sep 17 00:00:00 2001
From: Mathis De Witte <mathisdw@icloud.com>
Date: Mon, 20 May 2024 19:05:43 +0200
Subject: [PATCH 10/25] fix restrictions dialog

---
 .../RestrictionTemplateUI.tsx                 |  2 +-
 .../RestrictionsDialog.tsx                    | 20 ++-----------------
 2 files changed, 3 insertions(+), 19 deletions(-)

diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
index 070550be..857cc83f 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionTemplateUI.tsx
@@ -1,4 +1,4 @@
-import React, { useState } from 'react'
+import { useState } from 'react'
 import {
     Box,
     TextField,
diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionsDialog.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionsDialog.tsx
index 942d6d54..7a719850 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionsDialog.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/RestrictionsDialog.tsx
@@ -81,7 +81,6 @@ export default function RestrictionsDialog({
 }: RestrictionsDialogProps) {
     const [openTextEditor, setOpenTextEditor] = useState(false)
     const [openTemplateInterface, setOpenTemplateInterface] = useState(false)
-    const [mustPass, setMustPass] = useState(false)
     const [openTemplateInUI, setOpenTemplateInUI] = useState(false)
     const [textFieldContent, setTextFieldContent] = useState('')
     const [restrictionName, setRestrictionName] = useState('')
@@ -103,7 +102,7 @@ export default function RestrictionsDialog({
                 const newRestriction: restriction = {
                     script: file.name,
                     file: file,
-                    moet_slagen: mustPass,
+                    moet_slagen: false,
                 }
                 newRestrictions.push(newRestriction)
             }
@@ -126,7 +125,7 @@ export default function RestrictionsDialog({
                 restrictionName + restrictionType,
                 { type: 'text/plain' }
             ),
-            moet_slagen: mustPass,
+            moet_slagen: false,
         }
         setRestrictions([...restrictions, newRestriction])
         handleCloseTextEditor()
@@ -239,21 +238,6 @@ export default function RestrictionsDialog({
             >
                 {buttons}
             </Box>
-            <Box
-                paddingTop="5px"
-                display={'flex'}
-                flexDirection={'row'}
-                alignItems={'center'}
-            >
-                <Typography variant={'body2'}>
-                    {t('must_pass') + ':'}
-                </Typography>
-                <Switch
-                    id="mustPassSwitch"
-                    value={mustPass}
-                    onChange={() => setMustPass(!mustPass)}
-                />
-            </Box>
             <Box padding="20px" />
             <Typography variant={'h6'} color={'secondary.contrastText'}>
                 {t('choose_existing') + ':'}

From a88b7fa3e5c00893de7d4ee8a19577c214d8e9d5 Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 19:20:35 +0200
Subject: [PATCH 11/25] groepsleden knop geeft nu geen error meer, maar laat
 weten aan de student dat die nog niet in een groepje zit

---
 .../pages/assignmentPage/AssignmentPage.tsx   | 18 ++++---
 .../src/pages/subjectsPage/StudentPopUp.tsx   | 48 ++++++++++++-------
 .../src/pages/subjectsPage/SubjectsPage.tsx   |  1 +
 3 files changed, 43 insertions(+), 24 deletions(-)

diff --git a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
index fe9fb20f..a532ef89 100644
--- a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
+++ b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
@@ -187,15 +187,18 @@ export function AssignmentPage() {
             const groupResponse = await instance.get(
                 `groepen/?student=${user.user}&project=${assignmentId}`
             )
-            const group: Group = groupResponse.data[0]
 
-            const studentPromises: Promise<AxiosResponse<User>>[] =
-                group.studenten.map((id: number) =>
-                    instance.get('/gebruikers/' + id)
-                )
+            if (groupResponse.data.length > 0) {
+                const group: Group = groupResponse.data[0]
+
+                const studentPromises: Promise<AxiosResponse<User>>[] =
+                    group.studenten.map((id: number) =>
+                        instance.get('/gebruikers/' + id)
+                    )
 
-            const temp_students = await axios.all(studentPromises)
-            setStudents(temp_students.map((res) => res.data))
+                const temp_students = await axios.all(studentPromises)
+                setStudents(temp_students.map((res) => res.data))
+            }
 
             setStudentsLoading(false)
         }
@@ -837,6 +840,7 @@ export function AssignmentPage() {
                                                                             : students
                                                                     }
                                                                     text="group_members"
+                                                                    noGroup={students.length == 0}
                                                                 />
                                                             </>
                                                         )}
diff --git a/frontend/frontend/src/pages/subjectsPage/StudentPopUp.tsx b/frontend/frontend/src/pages/subjectsPage/StudentPopUp.tsx
index cc728f50..f045e495 100644
--- a/frontend/frontend/src/pages/subjectsPage/StudentPopUp.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/StudentPopUp.tsx
@@ -17,9 +17,10 @@ import { t } from 'i18next'
 interface StudentPopUpProps {
     students: User[]
     text: string
+    noGroup: boolean
 }
 
-export default function StudentPopUp({ students, text }: StudentPopUpProps) {
+export default function StudentPopUp({ students, text, noGroup }: StudentPopUpProps) {
     const [open, setOpen] = React.useState(false)
 
     const handleClose = () => {
@@ -59,22 +60,35 @@ export default function StudentPopUp({ students, text }: StudentPopUpProps) {
                     </IconButton>
                 </DialogTitle>
                 <DialogContent dividers>
-                    {/* List of Students */}
-                    {students.length > 0 ? (
-                        <List>
-                            {students.map((student) => (
-                                <ListItem key={student.user}>
-                                    <ListItemText
-                                        primary={`${student.first_name} ${student.last_name}`}
-                                    />
-                                </ListItem>
-                            ))}
-                        </List>
-                    ) : (
-                        <Typography variant="body1">
-                            {t('loading') + ' ' + t('students') + '...'}
-                        </Typography>
-                    )}
+                    {noGroup ?
+                        <>
+                            <Typography variant="body1">
+                                {t('noGroup')}
+                            </Typography>
+                            <Typography variant="body1">
+                                {t('contactTeacher')}
+                            </Typography>
+                        </>
+                    :
+                        <>
+                            {/* List of Students */}
+                            {students.length > 0 ? (
+                                <List>
+                                    {students.map((student) => (
+                                        <ListItem key={student.user}>
+                                            <ListItemText
+                                                primary={`${student.first_name} ${student.last_name}`}
+                                            />
+                                        </ListItem>
+                                    ))}
+                                </List>
+                            ) : (
+                                <Typography variant="body1">
+                                    {t('loading') + ' ' + t('students') + '...'}
+                                </Typography>
+                            )}
+                        </>
+                    }
                 </DialogContent>
             </Dialog>
         </>
diff --git a/frontend/frontend/src/pages/subjectsPage/SubjectsPage.tsx b/frontend/frontend/src/pages/subjectsPage/SubjectsPage.tsx
index 83f6e456..86909b4b 100644
--- a/frontend/frontend/src/pages/subjectsPage/SubjectsPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/SubjectsPage.tsx
@@ -455,6 +455,7 @@ export function SubjectsPage() {
                                             studentsLoading ? [] : students
                                         }
                                         text="students"
+                                        noGroup={false}
                                     ></StudentPopUp>
                                 </Box>
                             </Stack>

From 6c9d245354dfa1adf53f76ce546ba7c42f1ddc4c Mon Sep 17 00:00:00 2001
From: Mathis De Witte <mathisdw@icloud.com>
Date: Mon, 20 May 2024 19:38:35 +0200
Subject: [PATCH 12/25] headers for restrictions list

---
 .../AddChangeAssignmentPage.tsx                  | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
index 39f49c20..467c0e7d 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
@@ -805,6 +805,22 @@ export function AddChangeAssignmentPage() {
                                                 {t('restrictions')}
                                             </Typography>
                                             <Box sx={{ padding: 1 }}>
+                                                <Box
+                                                    display="flex"
+                                                    flexDirection="row"
+                                                    justifyContent="space-between"
+                                                    sx={{ padding: 1 }}
+                                                >
+                                                    <Typography variant="body1">
+                                                        {t('name')}
+                                                    </Typography>
+                                                    <Typography variant="body1">
+                                                        {t('must_pass')}
+                                                    </Typography>
+                                                    <Typography variant="body1">
+                                                        {t('remove')}
+                                                    </Typography>
+                                                </Box>
                                                 {/*This list will render the restrictions that are added to the assignment.*/}
                                                 <List
                                                     sx={{

From 7ba0357f1383333b1b63e0b01e5e167cfc02328a Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 19:51:54 +0200
Subject: [PATCH 13/25] opdracht icons worden nu wel geupdate op verandering
 van jaar in kalender

---
 frontend/frontend/src/components/DeadlineCalendar.tsx | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/frontend/frontend/src/components/DeadlineCalendar.tsx b/frontend/frontend/src/components/DeadlineCalendar.tsx
index f5007ba6..9835c497 100644
--- a/frontend/frontend/src/components/DeadlineCalendar.tsx
+++ b/frontend/frontend/src/components/DeadlineCalendar.tsx
@@ -215,6 +215,12 @@ export function DeadlineCalendar({
         fetchHighlightedDays(date)
     }
 
+    const handleYearChange = (date: Dayjs) => {
+        setIsLoading(true)
+        setHighlightedDays([])
+        fetchHighlightedDays(date)
+    }
+
     return (
         <>
             <Stack direction={'column'}>
@@ -224,6 +230,7 @@ export function DeadlineCalendar({
                     value={value}
                     onChange={(newValue) => setValue(newValue)}
                     onMonthChange={(newValue) => handleMonthChange(newValue)}
+                    onYearChange={(newValue) => handleYearChange(newValue)}
                     renderLoading={() => <DayCalendarSkeleton />}
                     loading={isLoading}
                     sx={dateStyle}

From 22e6eb23d7d0b8f32e13a356d9d04313c083c13e Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 19:55:46 +0200
Subject: [PATCH 14/25] status op mainpage in orde gebracht

---
 .../frontend/src/components/CourseCard.tsx    | 61 ++++++++++++++++---
 .../src/pages/mainPage/ArchivedView.tsx       |  3 +
 .../src/pages/mainPage/CoursesView.tsx        |  3 +
 .../frontend/src/pages/mainPage/MainPage.tsx  |  2 +
 4 files changed, 59 insertions(+), 10 deletions(-)

diff --git a/frontend/frontend/src/components/CourseCard.tsx b/frontend/frontend/src/components/CourseCard.tsx
index 0359ee2c..75d9bde7 100644
--- a/frontend/frontend/src/components/CourseCard.tsx
+++ b/frontend/frontend/src/components/CourseCard.tsx
@@ -19,6 +19,9 @@ import { Course, project } from '../pages/mainPage/MainPage.tsx'
 import dayjs from 'dayjs'
 
 import { CourseCardSkeleton } from './CourseCardSkeleton.tsx'
+import { Submission } from '../pages/submissionPage/SubmissionPage.tsx'
+import { Group } from '../pages/groupsPage/GroupsPage.tsx'
+import axios from 'axios'
 /*
  * CourseCard component displays a card with course information and a list of assignments
  * @param courseId: string, the id of the course
@@ -29,6 +32,7 @@ import { CourseCardSkeleton } from './CourseCardSkeleton.tsx'
 //TODO: fix archived with state so that the card moves to ArchivedView when archived
 
 interface CourseCardProps {
+    userid: number
     courseId: string
     archived: boolean
     isStudent: boolean
@@ -38,6 +42,7 @@ interface CourseCardProps {
 }
 
 export function CourseCard({
+    userid,
     courseId,
     archived,
     isStudent,
@@ -54,6 +59,9 @@ export function CourseCard({
         gearchiveerd: false,
     })
     const [assignments, setAssignments] = useState<project[]>([])
+    const [groupsWithSubmissions, setGroupsWithSubmissions] = useState<Group[]>(
+        []
+    )
     const [teachers, setTeachers] = useState<
         { first_name: string; last_name: string }[]
     >([])
@@ -77,6 +85,32 @@ export function CourseCard({
                 setCourse(courseResponse.data)
                 setAssignments(assignmentsResponse.data)
 
+                //fetch submissions as well if user is student
+                if (isStudent) {
+                    const groups: Group[] = await instance
+                        .get(`/groepen/?student=${userid}`)
+                        .then((response) => response.data)
+                    const submissionPromises = groups.map(async (group) => {
+                        const response = await instance.get<Submission>(
+                            `/indieningen/?project=${group.project}`
+                        )
+                        return response.data
+                    })
+                    const submissions = await axios.all(submissionPromises)
+
+                    console.log(groups)
+                    const validGroups = groups.filter((group) =>
+                        submissions
+                            .flat(Infinity)
+                            .some(
+                                (submission) =>
+                                    submission.groep === group.groep_id
+                            )
+                    )
+
+                    setGroupsWithSubmissions(validGroups)
+                }
+
                 if (courseResponse.data && courseResponse.data.lesgevers) {
                     const lesgevers = []
                     for (const teacherId of courseResponse.data.lesgevers) {
@@ -346,10 +380,11 @@ export function CourseCard({
                                                                 'DD/MM/YYYY HH:mm'
                                                             ) || undefined
                                                         }
-                                                        status={
-                                                            assignment.project_id ===
-                                                            1
-                                                        } //TODO dit moet nog aangepast worden
+                                                        status={groupsWithSubmissions.some(
+                                                            (group) =>
+                                                                group.project ===
+                                                                assignment.project_id
+                                                        )}
                                                         isStudent={isStudent}
                                                     />
                                                 ))}
@@ -386,8 +421,13 @@ export function CourseCard({
                                                                     undefined
                                                                 }
                                                                 status={
-                                                                    assignment.project_id ===
-                                                                    1
+                                                                    groupsWithSubmissions.some(
+                                                                        (
+                                                                            group
+                                                                        ) =>
+                                                                            group.project ===
+                                                                            assignment.project_id
+                                                                    )
                                                                     //TODO: status has to check if there is already a submission
                                                                 }
                                                                 isStudent={
@@ -429,10 +469,11 @@ export function CourseCard({
                                                                     ) ||
                                                                     undefined
                                                                 }
-                                                                status={
-                                                                    assignment.project_id ===
-                                                                    1
-                                                                }
+                                                                status={groupsWithSubmissions.some(
+                                                                    (group) =>
+                                                                        group.project ===
+                                                                        assignment.project_id
+                                                                )}
                                                                 isStudent={
                                                                     isStudent
                                                                 }
diff --git a/frontend/frontend/src/pages/mainPage/ArchivedView.tsx b/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
index 616165b6..5997915a 100644
--- a/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
+++ b/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
@@ -3,6 +3,7 @@ import { CourseCard } from '../../components/CourseCard.tsx'
 import { Course } from './MainPage.tsx'
 
 interface CourseCardProps {
+    userid: number
     isStudent: boolean
     archivedCourses: Course[]
     pinnedCourses: number[]
@@ -10,6 +11,7 @@ interface CourseCardProps {
 }
 
 export function ArchivedView({
+    userid,
     isStudent,
     archivedCourses,
     pinnedCourses,
@@ -40,6 +42,7 @@ export function ArchivedView({
                         {archivedCourses.map((course) => {
                             return (
                                 <CourseCard
+                                    userid={userid}
                                     courseId={course.vak_id.toString()}
                                     archived={true}
                                     isStudent={isStudent}
diff --git a/frontend/frontend/src/pages/mainPage/CoursesView.tsx b/frontend/frontend/src/pages/mainPage/CoursesView.tsx
index 5660a1cf..ce262aca 100644
--- a/frontend/frontend/src/pages/mainPage/CoursesView.tsx
+++ b/frontend/frontend/src/pages/mainPage/CoursesView.tsx
@@ -5,6 +5,7 @@ import { useNavigate } from 'react-router-dom'
 import { Course } from './MainPage.tsx'
 
 interface CourseCardProps {
+    userid: number
     isStudent: boolean
     activecourses: Course[]
     pinnedCourses: number[]
@@ -13,6 +14,7 @@ interface CourseCardProps {
 }
 
 export function CoursesView({
+    userid,
     isStudent,
     activecourses,
     pinnedCourses,
@@ -46,6 +48,7 @@ export function CoursesView({
                         A CourseCard displays brief information about the course such as the title, deadlines, ...*/}
                         {activecourses.map((course: Course) => (
                             <CourseCard
+                                userid={userid}
                                 key={course.naam}
                                 courseId={course.vak_id.toString()}
                                 archived={false}
diff --git a/frontend/frontend/src/pages/mainPage/MainPage.tsx b/frontend/frontend/src/pages/mainPage/MainPage.tsx
index b02a936f..6f99df8e 100644
--- a/frontend/frontend/src/pages/mainPage/MainPage.tsx
+++ b/frontend/frontend/src/pages/mainPage/MainPage.tsx
@@ -228,6 +228,7 @@ export default function MainPage() {
                                   ]
                                 : [
                                       <CoursesView
+                                          userid={user}
                                           isStudent={role == 'student'}
                                           activecourses={courses
                                               .filter(
@@ -273,6 +274,7 @@ export default function MainPage() {
                                           pinCourse={pinCourse}
                                       />,
                                       <ArchivedView
+                                          userid={user}
                                           isStudent={role == 'student'}
                                           archivedCourses={courses
                                               .filter(

From 87b2811c6af6e08df45782e0d7b2e6aab3c1bbd4 Mon Sep 17 00:00:00 2001
From: Alexandre Paice <alexandre.paice@ugent.be>
Date: Mon, 20 May 2024 20:10:00 +0200
Subject: [PATCH 15/25] studentPopUp fixed

---
 frontend/frontend/src/pages/submissionPage/SubmissionPage.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/frontend/frontend/src/pages/submissionPage/SubmissionPage.tsx b/frontend/frontend/src/pages/submissionPage/SubmissionPage.tsx
index 0654ade6..a2a7cb76 100644
--- a/frontend/frontend/src/pages/submissionPage/SubmissionPage.tsx
+++ b/frontend/frontend/src/pages/submissionPage/SubmissionPage.tsx
@@ -298,6 +298,7 @@ export function SubmissionPage() {
                             <StudentPopUp
                                 students={studentsLoading ? [] : students}
                                 text="group_members"
+                                noGroup={false}
                             />
                         )}
                     </Box>

From bea5be1fe57c761dc51c0fc400954ca791589b31 Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 20:24:08 +0200
Subject: [PATCH 16/25] pin icon verschijnt rechtsboven

---
 frontend/frontend/src/components/CourseCard.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/frontend/src/components/CourseCard.tsx b/frontend/frontend/src/components/CourseCard.tsx
index 75d9bde7..fe65304f 100644
--- a/frontend/frontend/src/components/CourseCard.tsx
+++ b/frontend/frontend/src/components/CourseCard.tsx
@@ -232,10 +232,6 @@ export function CourseCard({
                                 </Box>
                                 {/* Number of students enrolled */}
                                 <Box>
-                                    <Typography variant={'subtitle1'}>
-                                        {t('students') + ': '}
-                                        {course.studenten.length || 0}
-                                    </Typography>
                                     <Box
                                         display={'flex'}
                                         flexDirection={'column'}
@@ -261,6 +257,10 @@ export function CourseCard({
                                             )}
                                         </IconButton>
                                     </Box>
+                                    <Typography variant={'subtitle1'}>
+                                        {t('students') + ': '}
+                                        {course.studenten.length || 0}
+                                    </Typography>
                                 </Box>
                             </Box>
                         </CardActionArea>

From afd8c575a65cffc6b947ec2e12d247b3bb129a02 Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 20:53:37 +0200
Subject: [PATCH 17/25] 'geen deadline' word nu overal correct getoond

---
 .../frontend/src/components/CourseCard.tsx    | 25 ++++---
 .../pages/assignmentPage/AssignmentPage.tsx   | 68 ++++++++-----------
 .../AssignmentListItemSubjectsPage.tsx        |  2 +-
 .../src/pages/subjectsPage/ProjectsView.tsx   | 12 ++--
 4 files changed, 51 insertions(+), 56 deletions(-)

diff --git a/frontend/frontend/src/components/CourseCard.tsx b/frontend/frontend/src/components/CourseCard.tsx
index fe65304f..52af7594 100644
--- a/frontend/frontend/src/components/CourseCard.tsx
+++ b/frontend/frontend/src/components/CourseCard.tsx
@@ -374,11 +374,13 @@ export function CourseCard({
                                                             assignment.titel
                                                         }
                                                         dueDate={
-                                                            dayjs(
-                                                                assignment.deadline
-                                                            ).format(
-                                                                'DD/MM/YYYY HH:mm'
-                                                            ) || undefined
+                                                            assignment.deadline
+                                                                ? dayjs(
+                                                                    assignment.deadline
+                                                                  ).format(
+                                                                    'DD/MM/YYYY HH:mm'
+                                                                  )
+                                                                : undefined
                                                         }
                                                         status={groupsWithSubmissions.some(
                                                             (group) =>
@@ -413,12 +415,13 @@ export function CourseCard({
                                                                     assignment.titel
                                                                 }
                                                                 dueDate={
-                                                                    dayjs(
-                                                                        assignment.deadline
-                                                                    ).format(
-                                                                        'DD/MM/YYYY HH:mm'
-                                                                    ) ||
-                                                                    undefined
+                                                                    assignment.deadline
+                                                                        ? dayjs(
+                                                                            assignment.deadline
+                                                                          ).format(
+                                                                            'DD/MM/YYYY HH:mm'
+                                                                          )
+                                                                        : undefined
                                                                 }
                                                                 status={
                                                                     groupsWithSubmissions.some(
diff --git a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
index a532ef89..ffa4f0cd 100644
--- a/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
+++ b/frontend/frontend/src/pages/assignmentPage/AssignmentPage.tsx
@@ -410,26 +410,20 @@ export function AssignmentPage() {
                                                 height={50}
                                             />
                                         ) : (
-                                            <>
-                                                {assignment !== undefined &&
-                                                    assignment.deadline !==
-                                                        null && (
-                                                        <Typography
-                                                            variant="h6"
-                                                            color={
-                                                                'text.primary'
-                                                            }
-                                                        >
-                                                            {assignment
-                                                                ? dayjs(
-                                                                      assignment.deadline
-                                                                  ).format(
-                                                                      'DD/MM/YYYY HH:mm'
-                                                                  )
-                                                                : 'no deadline'}
-                                                        </Typography>
-                                                    )}
-                                            </>
+                                            <Typography
+                                                variant="h6"
+                                                color={
+                                                    'text.primary'
+                                                }
+                                            >
+                                                {assignment && assignment.deadline
+                                                    ? dayjs(
+                                                            assignment.deadline
+                                                        ).format(
+                                                            'DD/MM/YYYY HH:mm'
+                                                        )
+                                                    : t('no_deadline')}
+                                            </Typography>
                                         )}
                                     </Box>
                                     {loading ? (
@@ -761,26 +755,20 @@ export function AssignmentPage() {
                                                     height={50}
                                                 />
                                             ) : (
-                                                <>
-                                                    {assignment !== undefined &&
-                                                        assignment.deadline !==
-                                                            null && (
-                                                            <Typography
-                                                                variant="h6"
-                                                                color={
-                                                                    'text.primary'
-                                                                }
-                                                            >
-                                                                {assignment
-                                                                    ? dayjs(
-                                                                          assignment.deadline
-                                                                      ).format(
-                                                                          'DD/MM/YYYY HH:mm'
-                                                                      )
-                                                                    : 'no deadline'}
-                                                            </Typography>
-                                                        )}
-                                                </>
+                                                <Typography
+                                                    variant="h6"
+                                                    color={
+                                                        'text.primary'
+                                                    }
+                                                >
+                                                    {assignment && assignment.deadline
+                                                        ? dayjs(
+                                                                assignment.deadline
+                                                            ).format(
+                                                                'DD/MM/YYYY HH:mm'
+                                                            )
+                                                        : t('no_deadline')}
+                                                </Typography>
                                             )}
                                         </Box>
                                         <Box style={{ flexGrow: 1 }} />
diff --git a/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx b/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
index 66c2570a..11fd7e66 100644
--- a/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AssignmentListItemSubjectsPage.tsx
@@ -31,7 +31,7 @@ import { EvenlySpacedRow } from '../../components/CustomComponents.tsx'
 
 interface AssignmentListItemSubjectsPageProps {
     projectName: string
-    dueDate: Dayjs
+    dueDate: Dayjs | undefined
     submissions: number
     score: Score | undefined
     maxScore: number
diff --git a/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx b/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
index caa0953d..c40f2be5 100644
--- a/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/ProjectsView.tsx
@@ -271,10 +271,14 @@ export function ProjectsView({
                                                     projectName={
                                                         project.assignment.titel
                                                     }
-                                                    dueDate={dayjs(
-                                                        project.assignment
-                                                            .deadline
-                                                    )}
+                                                    dueDate={
+                                                        project.assignment.deadline
+                                                            ? dayjs(
+                                                              project.assignment
+                                                                .deadline
+                                                              )
+                                                            : undefined
+                                                    }
                                                     submissions={
                                                         project.submissions
                                                             ? project.submissions

From 423ba47ea826f1fe5398a9c4e47eab9cdabb79ad Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 21:05:06 +0200
Subject: [PATCH 18/25] verwijderen restricties mogelijk + toevoegknop
 verplaatst

---
 .../AddChangeAssignmentPage.tsx               | 76 ++++++++++++-------
 1 file changed, 47 insertions(+), 29 deletions(-)

diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
index 467c0e7d..65eef916 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
@@ -95,6 +95,7 @@ export function AddChangeAssignmentPage() {
     const [description, setDescription] = useState('')
     const [dueDate, setDueDate] = useState<Dayjs | null>(null)
     const [extraDueDate, setExtraDueDate] = useState<Dayjs | null>(null)
+    const [oldRestrictions, setOldRestrictions] = useState<restriction[]>([])
     const [restrictions, setRestrictions] = useState<restriction[]>([])
     const [visible, setVisible] = useState(false)
     const [assignmentFile, setAssignmentFile] = useState<File>()
@@ -232,6 +233,7 @@ export function AddChangeAssignmentPage() {
                                 console.error(error)
                             })
                     }
+                    setOldRestrictions(restrictions)
                     setRestrictions(restrictions)
                 })
                 .catch((error) => {
@@ -332,6 +334,19 @@ export function AddChangeAssignmentPage() {
             },
         }
 
+        //delete removed restrictions
+        oldRestrictions.forEach((oldRestriction) => {
+            if (!restrictions.includes(oldRestriction)) {
+                instance
+                    .delete(
+                        '/restricties/' + oldRestriction.restrictie_id + '/'
+                    )
+                    .catch((error) => {
+                        console.error(error)
+                    })
+            }
+        })
+
         restrictions.forEach((restriction) => {
             const formData = new FormData()
             formData.append('project', projectId)
@@ -732,8 +747,8 @@ export function AddChangeAssignmentPage() {
                                     <Card aria-label={'description'}>
                                         <Box
                                             padding={2}
-                                            maxHeight={'20svh'}
-                                            minHeight={'20svh'}
+                                            maxHeight={'18svh'}
+                                            minHeight={'18svh'}
                                         >
                                             <Typography
                                                 variant={'h5'}
@@ -776,7 +791,7 @@ export function AddChangeAssignmentPage() {
                                                     }
                                                     sx={{
                                                         overflowY: 'auto',
-                                                        maxHeight: '25svh',
+                                                        maxHeight: '23svh',
                                                     }}
                                                 />
                                             )}
@@ -788,6 +803,8 @@ export function AddChangeAssignmentPage() {
                                         marginTop={3}
                                         display={'flex'}
                                         flexDirection={'row'}
+                                        alignItems={'flex-end'}
+                                        gap={1}
                                     >
                                         <Card
                                             sx={{
@@ -864,34 +881,35 @@ export function AddChangeAssignmentPage() {
                                                     )}
                                                 </List>
                                             </Box>
-                                            <Box
-                                                width={'100%'}
-                                                display={'flex'}
-                                                justifyContent={'flex-end'}
+                                        </Card>
+                                        <Box
+                                            height={'fit-content'}
+                                            width={'fit-content'}
+                                            display={'flex'}
+                                            flexDirection={'column'}
+                                            justifyContent={'flex-end'}
+                                            alignItems={'flex-end'}
+                                        >
+                                            <Tooltip
+                                                title={t('add_restriction')}
                                             >
-                                                <Tooltip
-                                                    title={t('add_restriction')}
-                                                >
-                                                    <AddRestrictionButton
-                                                        // When this button is clicked, a pop up will show.
-                                                        // This popup will allow you to choose to make a restriction yourself,
-                                                        // create one starting from a template,
-                                                        // or choose a file from the system.
-                                                        restrictions={
-                                                            restrictions
-                                                        }
-                                                        setRestrictions={(
+                                                <AddRestrictionButton
+                                                    // When this button is clicked, a pop up will show.
+                                                    // This popup will allow you to choose to make a restriction yourself,
+                                                    // create one starting from a template,
+                                                    // or choose a file from the system.
+                                                    restrictions={restrictions}
+                                                    setRestrictions={(
+                                                        newRestrictions
+                                                    ) =>
+                                                        setRestrictions(
                                                             newRestrictions
-                                                        ) =>
-                                                            setRestrictions(
-                                                                newRestrictions
-                                                            )
-                                                        }
-                                                        userid={user.user}
-                                                    ></AddRestrictionButton>
-                                                </Tooltip>
-                                            </Box>
-                                        </Card>
+                                                        )
+                                                    }
+                                                    userid={user.user}
+                                                ></AddRestrictionButton>
+                                            </Tooltip>
+                                        </Box>
                                     </Box>
                                     {/* Main actions section */}
                                     <Box

From 943f6e389976bc8cadc4af610a489a56a35d73df Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 21:33:24 +0200
Subject: [PATCH 19/25] patch fix

---
 .../addChangeAssignmentPage/AddChangeAssignmentPage.tsx    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
index 65eef916..bc619a6d 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
@@ -350,9 +350,7 @@ export function AddChangeAssignmentPage() {
         restrictions.forEach((restriction) => {
             const formData = new FormData()
             formData.append('project', projectId)
-            if (restriction.file !== undefined) {
-                formData.append('script', restriction.file)
-            }
+
             formData.append('moet_slagen', restriction.moet_slagen.toString())
             if (restriction.restrictie_id !== undefined) {
                 formData.append(
@@ -369,6 +367,9 @@ export function AddChangeAssignmentPage() {
                         console.error(error)
                     })
             } else {
+                if (restriction.file !== undefined) {
+                    formData.append('script', restriction.file)
+                }
                 instance
                     .post('/restricties/', formData, config)
                     .catch((error) => {

From 259e4ec9c99dc532d1781805e16c8dce5374a7c9 Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 21:52:46 +0200
Subject: [PATCH 20/25] voeg jezelf toe bij nieuw vak, geen spookvakken meer

---
 frontend/frontend/src/components/Header.tsx            |  2 +-
 .../src/pages/subjectsPage/AddChangeSubjectPage.tsx    | 10 +++++++++-
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/frontend/frontend/src/components/Header.tsx b/frontend/frontend/src/components/Header.tsx
index 0a21c2b3..f8a83cd3 100644
--- a/frontend/frontend/src/components/Header.tsx
+++ b/frontend/frontend/src/components/Header.tsx
@@ -147,7 +147,7 @@ export const Header = ({ variant, title }: Props) => {
                         >
                             {variant === 'main' && (
                                 <Box
-                                    id='logo'
+                                    id="logo"
                                     component="img"
                                     src={'assets/logo_duif_top_wit.png'}
                                     alt="logo_app"
diff --git a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
index 2d33ca2a..b070d9c0 100644
--- a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
@@ -79,7 +79,12 @@ function UserList(
                             return (
                                 <>
                                     <Divider />
-                                    <ListItem>
+                                    <ListItem
+                                        disabled={
+                                            users.length == 1 &&
+                                            users[0].is_lesgever
+                                        }
+                                    >
                                         <EvenlySpacedRow
                                             items={[
                                                 <ListItemText
@@ -469,6 +474,9 @@ export function AddChangeSubjectPage() {
         async function fetchUser() {
             setUserLoading(true)
             const userResponse = await instance.get('/gebruikers/me/')
+            if (vakID == undefined) {
+                setTeachers([userResponse.data])
+            }
             setUser(userResponse.data)
             setUserLoading(false)
         }

From f5727c674ea0fe2b89da629c12313acfaedb033a Mon Sep 17 00:00:00 2001
From: elias <Elias.Meynckens@UGent.be>
Date: Mon, 20 May 2024 21:59:20 +0200
Subject: [PATCH 21/25] cleanup restrictions box in addchangeAssigmentPage

---
 .../src/components/RestrictionCard.tsx        | 17 ++---
 .../AddChangeAssignmentPage.tsx               | 74 ++++++++++---------
 2 files changed, 46 insertions(+), 45 deletions(-)

diff --git a/frontend/frontend/src/components/RestrictionCard.tsx b/frontend/frontend/src/components/RestrictionCard.tsx
index cdfa256b..75ad36ee 100644
--- a/frontend/frontend/src/components/RestrictionCard.tsx
+++ b/frontend/frontend/src/components/RestrictionCard.tsx
@@ -2,6 +2,7 @@ import { restriction } from '../pages/addChangeAssignmentPage/AddChangeAssignmen
 import { Box, IconButton, Switch, Typography } from '@mui/material'
 import CloseIcon from '@mui/icons-material/Close'
 import React from 'react'
+import {EvenlySpacedRow} from "./CustomComponents.tsx";
 
 interface RestrictionCardProps {
     restriction: restriction
@@ -28,26 +29,20 @@ export function RestrictionCard({
 
     return (
         <>
-            <Box
-                display={'flex'}
-                flexDirection={'row'}
-                width={'100%'}
-                justifyContent={'space-between'}
-            >
+            <EvenlySpacedRow items={[
                 <Typography id="script" variant={'body2'}>
                     {restriction.script.replace(/^.*[\\/]/, '')}
-                </Typography>
+                </Typography>,
                 <Switch
                     id="mustPassSwitch"
                     value={mustPass}
                     checked={mustPass}
                     onChange={() => handleMustPassChange()}
-                />
-
+                />,
                 <IconButton id="closeButton" onClick={handleRemove}>
                     <CloseIcon />
-                </IconButton>
-            </Box>
+                </IconButton>]}
+            />
         </>
     )
 }
diff --git a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
index 65eef916..ac17ca1a 100644
--- a/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
+++ b/frontend/frontend/src/pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx
@@ -1,4 +1,4 @@
-import { Card } from '../../components/CustomComponents.tsx'
+import {Card, Divider, EvenlySpacedRow} from '../../components/CustomComponents.tsx'
 import {
     Box,
     CircularProgress,
@@ -538,7 +538,7 @@ export function AddChangeAssignmentPage() {
                                             ) : (
                                                 <TextField
                                                     type="text"
-                                                    placeholder={'Title'}
+                                                    placeholder={t('name')}
                                                     error={
                                                         assignmentErrors.title
                                                     }
@@ -808,37 +808,40 @@ export function AddChangeAssignmentPage() {
                                     >
                                         <Card
                                             sx={{
-                                                padding: 1,
                                                 backgroundColor:
                                                     'background.default',
                                                 width: '70%',
                                                 height: '28svh',
                                             }}
                                         >
+                                            <Box
+                                                sx={{
+                                                    backgroundColor: 'secondary.main',
+                                                    height: 48,
+                                                    padding: 1,
+                                                }}
+                                            >
                                             <Typography
                                                 variant={'h5'}
                                                 fontWeight={'bold'}
                                             >
                                                 {t('restrictions')}
                                             </Typography>
-                                            <Box sx={{ padding: 1 }}>
-                                                <Box
-                                                    display="flex"
-                                                    flexDirection="row"
-                                                    justifyContent="space-between"
-                                                    sx={{ padding: 1 }}
-                                                >
-                                                    <Typography variant="body1">
+                                            <EvenlySpacedRow items={[
+                                                    <Typography variant="body1" fontWeight={'bold'}>
                                                         {t('name')}
-                                                    </Typography>
-                                                    <Typography variant="body1">
+                                                    </Typography>,
+                                                    <Typography variant="body1" fontWeight={'bold'}>
                                                         {t('must_pass')}
-                                                    </Typography>
-                                                    <Typography variant="body1">
+                                                    </Typography>,
+                                                    <Typography variant="body1" fontWeight={'bold'}>
                                                         {t('remove')}
-                                                    </Typography>
-                                                </Box>
-                                                {/*This list will render the restrictions that are added to the assignment.*/}
+                                                    </Typography>]}
+                                            />
+                                            </Box>
+                                            <Divider/>
+                                            {/*This list will render the restrictions that are added to the assignment.*/}
+                                            <Box sx={{marginTop: -1.1}}>
                                                 <List
                                                     sx={{
                                                         maxHeight: '18vh',
@@ -857,23 +860,26 @@ export function AddChangeAssignmentPage() {
                                                                     index
                                                                 ) => {
                                                                     return (
-                                                                        <ListItem
-                                                                            key={
-                                                                                index
-                                                                            }
-                                                                        >
-                                                                            <RestrictionCard
-                                                                                restriction={
-                                                                                    restriction
-                                                                                }
-                                                                                restrictions={
-                                                                                    restrictions
-                                                                                }
-                                                                                setRestrictions={
-                                                                                    setRestrictions
+                                                                        <>
+                                                                            <Divider/>
+                                                                            <ListItem sx={{ maxHeight: '45px'}}
+                                                                                key={
+                                                                                    index
                                                                                 }
-                                                                            />
-                                                                        </ListItem>
+                                                                            >
+                                                                                <RestrictionCard
+                                                                                    restriction={
+                                                                                        restriction
+                                                                                    }
+                                                                                    restrictions={
+                                                                                        restrictions
+                                                                                    }
+                                                                                    setRestrictions={
+                                                                                        setRestrictions
+                                                                                    }
+                                                                                />
+                                                                            </ListItem>
+                                                                        </>
                                                                     )
                                                                 }
                                                             )}

From 68dc6dd50374ef8b089f368bc8354a9d18812b3e Mon Sep 17 00:00:00 2001
From: Alexandre Paice <alexandre.paice@ugent.be>
Date: Mon, 20 May 2024 22:04:10 +0200
Subject: [PATCH 22/25] vakken renderen op jaartal

---
 .../frontend/src/components/CourseCard.tsx    |  1 +
 .../frontend/src/pages/mainPage/MainPage.tsx  | 29 +++++++++++++++----
 2 files changed, 24 insertions(+), 6 deletions(-)

diff --git a/frontend/frontend/src/components/CourseCard.tsx b/frontend/frontend/src/components/CourseCard.tsx
index fe65304f..e8ab1a07 100644
--- a/frontend/frontend/src/components/CourseCard.tsx
+++ b/frontend/frontend/src/components/CourseCard.tsx
@@ -54,6 +54,7 @@ export function CourseCard({
     const [course, setCourse] = useState<Course>({
         vak_id: 0,
         naam: '',
+        jaartal: 0,
         studenten: [],
         lesgevers: [],
         gearchiveerd: false,
diff --git a/frontend/frontend/src/pages/mainPage/MainPage.tsx b/frontend/frontend/src/pages/mainPage/MainPage.tsx
index 6f99df8e..961fab27 100644
--- a/frontend/frontend/src/pages/mainPage/MainPage.tsx
+++ b/frontend/frontend/src/pages/mainPage/MainPage.tsx
@@ -1,6 +1,6 @@
 import { Header } from '../../components/Header.tsx'
 import { Button } from '../../components/CustomComponents.tsx'
-import { Box, Stack } from '@mui/material'
+import { Box, Stack, MenuItem, Select } from '@mui/material'
 import TabSwitcher from '../../components/TabSwitcher.tsx'
 import { ArchivedView } from './ArchivedView.tsx'
 import { CoursesView } from './CoursesView.tsx'
@@ -17,6 +17,7 @@ import WarningPopup from '../../components/WarningPopup.tsx'
 export interface Course {
     vak_id: number
     naam: string
+    jaartal: number
     studenten: number[]
     lesgevers: number[]
     gearchiveerd: boolean
@@ -50,15 +51,13 @@ export default function MainPage() {
     const [courseOrder, setCourseOrder] = useState<number[]>([])
     const [deadlines, setDeadlines] = useState<Dayjs[]>([])
     const [loading, setLoading] = useState<boolean>(true)
+    const [selectedYear, setSelectedYear] = useState<number>(dayjs().year())
 
     //navigator for routing
     const [assignments, setAssignments] = useState<project[]>([])
     const navigator = useNavigate()
 
     useEffect(() => {
-        console.log('requesting api')
-        //set loading to true every time the data is requested
-
         // Get the courses, their projects, and their respective deadlines + the role of the user
         async function fetchData() {
             //set loading to true every time the data is requested
@@ -233,7 +232,7 @@ export default function MainPage() {
                                           activecourses={courses
                                               .filter(
                                                   (course) =>
-                                                      !course.gearchiveerd
+                                                      !course.gearchiveerd && course.jaartal === selectedYear
                                               )
                                               .sort((a: Course, b: Course) => {
                                                   if (
@@ -321,7 +320,25 @@ export default function MainPage() {
                                   ]
                         }
                     />
-
+                    <Box
+                    display={'flex'}
+                    justifyContent={'center'}
+                    alignItems={'center'}
+                    mb={2}
+                >
+                    <Select
+                        value={selectedYear}
+                        onChange={(e) => setSelectedYear(e.target.value as number)}
+                        label="Select Academic Year"
+                        sx={{ minWidth: 150 }}
+                    >
+                        {[2022, 2023, 2024, 2025].map((year) => (
+                            <MenuItem key={year} value={year}>
+                                {year}
+                            </MenuItem>
+                        ))}
+                    </Select>
+                    </Box>
                     {/* Add a calendar to the right of the mainpage. */}
                     <Box
                         aria-label={'calendarView'}

From 9cb42df6a0c1e29a0e87f235f63824ef8e8c02aa Mon Sep 17 00:00:00 2001
From: Gus Vanpoucke <159711661+gusvanpoucke@users.noreply.github.com>
Date: Mon, 20 May 2024 22:32:44 +0200
Subject: [PATCH 23/25] er verschijnen nu alerts als je op de
 addchangesubjectpage ongeldige gebruikers probeert toe te voegen, en de input
 velden worden automatisch gecleart

---
 frontend/frontend/src/i18n/en.ts              |  4 +++
 frontend/frontend/src/i18n/nl.ts              |  4 +++
 .../subjectsPage/AddChangeSubjectPage.tsx     | 34 ++++++++++++++++---
 3 files changed, 37 insertions(+), 5 deletions(-)

diff --git a/frontend/frontend/src/i18n/en.ts b/frontend/frontend/src/i18n/en.ts
index aa4ef3bd..3be359c7 100644
--- a/frontend/frontend/src/i18n/en.ts
+++ b/frontend/frontend/src/i18n/en.ts
@@ -120,6 +120,10 @@ const english = {
     open_with_ui: 'Open with UI',
     choose_existing: 'Choose existing',
     make_new_script: 'Make new script',
+    cant_add_teachers_to_student_list: 'You cannot add teachers to the student list',
+    cant_add_students_to_teacher_list: 'You cannot add students to the teacher list',
+    cant_add_users_twice: 'You cannot add users to a subject twice',
+    this_user_doesnt_exist: 'This user does not exist',
 }
 
 export default english
diff --git a/frontend/frontend/src/i18n/nl.ts b/frontend/frontend/src/i18n/nl.ts
index cb7a9b09..a00800fc 100644
--- a/frontend/frontend/src/i18n/nl.ts
+++ b/frontend/frontend/src/i18n/nl.ts
@@ -120,6 +120,10 @@ const dutch = {
     open_with_ui: 'Open met UI',
     choose_existing: 'Kies een bestaand script',
     make_new_script: 'Maak nieuw script aan',
+    cant_add_teachers_to_student_list: 'Je kan geen leerkrachten aan de studenten lijst toevoegen',
+    cant_add_students_to_teacher_list: 'Je kan geen studenten aan de leerkrachten lijst toevoegen',
+    cant_add_users_twice: 'Je kan gebruikers geen twee keer toevoegen',
+    this_user_doesnt_exist: 'Deze gebruiker bestaat niet',
 }
 
 export default dutch
diff --git a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
index 2d33ca2a..f516fdc5 100644
--- a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
@@ -18,7 +18,7 @@ import {
     Typography,
 } from '@mui/material'
 import { Header } from '../../components/Header'
-import React, { ChangeEvent, useEffect, useState } from 'react'
+import React, { ChangeEvent, useEffect, useRef, useState } from 'react'
 import { useNavigate, useParams } from 'react-router-dom'
 import List from '@mui/material/List'
 import { t } from 'i18next'
@@ -144,7 +144,8 @@ function UploadPart(
     handleFileChange: (e: ChangeEvent<HTMLInputElement>) => void,
     setEmail: React.Dispatch<React.SetStateAction<string>>,
     handleAdd: () => void,
-    str: string
+    str: string,
+    textFieldRef: React.RefObject<HTMLInputElement>
 ) {
     return (
         <>
@@ -159,6 +160,7 @@ function UploadPart(
                                 onChange={(event) =>
                                     setEmail(event.target.value)
                                 }
+                                inputRef={textFieldRef}
                             />
                         </Box>
                         <Box>
@@ -226,6 +228,8 @@ export function AddChangeSubjectPage() {
     const [studentFile, setStudentFile] = useState<File>()
     const [teacherFile, setTeacherFile] = useState<File>()
     const [user, setUser] = useState<User>()
+    const studentRef = useRef<HTMLInputElement>(null)
+    const teacherRef = useRef<HTMLInputElement>(null)
 
     const [vakID, setVakID] = useState(params.courseId)
 
@@ -256,6 +260,7 @@ export function AddChangeSubjectPage() {
             .then((res) => {
                 setStudents((oldstudents) => {
                     if (res.data.length == 0) {
+                        alert(t('this_user_doesnt_exist'))
                         return oldstudents
                     }
                     return addUser(false, res.data[0], oldstudents)
@@ -266,6 +271,11 @@ export function AddChangeSubjectPage() {
             })
 
         handleUploadStudent()
+
+        if (studentRef.current) {
+            studentRef.current.value = '';
+            setEmailStudent('')
+        }
     }
 
     const handleStudentFileChange = (
@@ -347,6 +357,7 @@ export function AddChangeSubjectPage() {
                     //This is like this to prevent the same user being in the list twice
 
                     if (res.data.length == 0) {
+                        alert(t('this_user_doesnt_exist'))
                         return oldteachers
                     }
 
@@ -357,6 +368,11 @@ export function AddChangeSubjectPage() {
                 console.log(err)
             })
         handleUploadTeacher()
+
+        if (teacherRef.current) {
+            teacherRef.current.value = '';
+            setEmailTeacher('')
+        }
     }
 
     const handleTeacherFileChange = (
@@ -421,6 +437,11 @@ export function AddChangeSubjectPage() {
         let found = false
         const id = userData.user
         if (userData.is_lesgever != isLesgever) {
+            if (userData.is_lesgever){
+                alert(t('cant_add_teachers_to_student_list'))
+            } else {
+                alert(t('cant_add_students_to_teacher_list'))
+            }
             return olduser
         }
         for (const teacher of olduser) {
@@ -429,6 +450,7 @@ export function AddChangeSubjectPage() {
             }
         }
         if (found) {
+            alert(t('cant_add_users_twice'))
             return olduser
         } else {
             return [...olduser, userData]
@@ -684,14 +706,15 @@ export function AddChangeSubjectPage() {
                                                         handleStudentFileChange,
                                                         setEmailStudent,
                                                         handleAddStudent,
-                                                        t('upload_students')
+                                                        t('upload_students'),
+                                                        studentRef,
                                                     )}
                                                 </Box>
                                                 {DialogWindow(
                                                     handleCloseStudent,
                                                     openStudent,
                                                     handleRemoveStudent,
-                                                    t('delete_student')
+                                                    t('delete_student'),
                                                 )}
                                             </Stack>
                                         </Box>
@@ -733,7 +756,8 @@ export function AddChangeSubjectPage() {
                                                         handleTeacherFileChange,
                                                         setEmailTeacher,
                                                         handleAddTeacher,
-                                                        t('upload_teachers')
+                                                        t('upload_teachers'),
+                                                        teacherRef,
                                                     )}
                                                 </Box>
                                                 {DialogWindow(

From 2b7145e8e360432571004fedfd318f0bf2e52e6c Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 23:00:32 +0200
Subject: [PATCH 24/25] popup weggewerkt en jaarkeuze andere kleur

---
 .../src/pages/mainPage/ArchivedView.tsx       |  2 +-
 .../src/pages/mainPage/CoursesView.tsx        |  2 +-
 .../frontend/src/pages/mainPage/MainPage.tsx  | 58 ++++++++++++-------
 .../subjectsPage/AddChangeSubjectPage.tsx     | 44 +++++++-------
 4 files changed, 60 insertions(+), 46 deletions(-)

diff --git a/frontend/frontend/src/pages/mainPage/ArchivedView.tsx b/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
index 5997915a..5dec46d6 100644
--- a/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
+++ b/frontend/frontend/src/pages/mainPage/ArchivedView.tsx
@@ -21,7 +21,7 @@ export function ArchivedView({
         <>
             <Stack
                 flexDirection={{ xs: 'column-reverse', md: 'row' }}
-                minWidth={{ md: '62svw', lg: '73svw' }}
+                minWidth={{ md: '47svw', lg: '68svw' }}
             >
                 <Stack
                     direction={'column'}
diff --git a/frontend/frontend/src/pages/mainPage/CoursesView.tsx b/frontend/frontend/src/pages/mainPage/CoursesView.tsx
index ce262aca..d89dbd5e 100644
--- a/frontend/frontend/src/pages/mainPage/CoursesView.tsx
+++ b/frontend/frontend/src/pages/mainPage/CoursesView.tsx
@@ -27,7 +27,7 @@ export function CoursesView({
         <>
             <Stack
                 flexDirection={{ xs: 'column-reverse', md: 'row' }}
-                minWidth={{ md: '62svw', lg: '73svw' }}
+                minWidth={{ md: '47svw', lg: '68svw' }}
             >
                 <Stack
                     direction={'column'}
diff --git a/frontend/frontend/src/pages/mainPage/MainPage.tsx b/frontend/frontend/src/pages/mainPage/MainPage.tsx
index 961fab27..52a9b7f5 100644
--- a/frontend/frontend/src/pages/mainPage/MainPage.tsx
+++ b/frontend/frontend/src/pages/mainPage/MainPage.tsx
@@ -1,6 +1,6 @@
 import { Header } from '../../components/Header.tsx'
 import { Button } from '../../components/CustomComponents.tsx'
-import { Box, Stack, MenuItem, Select } from '@mui/material'
+import { Box, MenuItem, Select, Stack } from '@mui/material'
 import TabSwitcher from '../../components/TabSwitcher.tsx'
 import { ArchivedView } from './ArchivedView.tsx'
 import { CoursesView } from './CoursesView.tsx'
@@ -202,8 +202,8 @@ export default function MainPage() {
                                               md: 'row',
                                           }}
                                           minWidth={{
-                                              md: '60svw',
-                                              lg: '69svw',
+                                              md: '40svw',
+                                              lg: '49svw',
                                           }}
                                       >
                                           {[...Array(3)].map((_, index) => (
@@ -216,8 +216,8 @@ export default function MainPage() {
                                               md: 'row',
                                           }}
                                           minWidth={{
-                                              md: '60svw',
-                                              lg: '69svw',
+                                              md: '40svw',
+                                              lg: '49svw',
                                           }}
                                       >
                                           {[...Array(3)].map((_, index) => (
@@ -232,7 +232,9 @@ export default function MainPage() {
                                           activecourses={courses
                                               .filter(
                                                   (course) =>
-                                                      !course.gearchiveerd && course.jaartal === selectedYear
+                                                      !course.gearchiveerd &&
+                                                      course.jaartal ===
+                                                          selectedYear
                                               )
                                               .sort((a: Course, b: Course) => {
                                                   if (
@@ -321,23 +323,35 @@ export default function MainPage() {
                         }
                     />
                     <Box
-                    display={'flex'}
-                    justifyContent={'center'}
-                    alignItems={'center'}
-                    mb={2}
-                >
-                    <Select
-                        value={selectedYear}
-                        onChange={(e) => setSelectedYear(e.target.value as number)}
-                        label="Select Academic Year"
-                        sx={{ minWidth: 150 }}
+                        height={'100%'}
+                        display={'flex'}
+                        flexDirection={'column'}
+                        justifyContent={'center'}
+                        alignItems={'flex-start'}
+                        mb={2}
                     >
-                        {[2022, 2023, 2024, 2025].map((year) => (
-                            <MenuItem key={year} value={year}>
-                                {year}
-                            </MenuItem>
-                        ))}
-                    </Select>
+                        <Select
+                            variant={'outlined'}
+                            color={'primary'}
+                            value={selectedYear}
+                            onChange={(e) =>
+                                setSelectedYear(e.target.value as number)
+                            }
+                            label="Select Academic Year"
+                            sx={{
+                                minWidth: 150,
+                                '& .MuiSelect-outlined': {
+                                    border: 1.5,
+                                    borderColor: 'primary.main',
+                                },
+                            }}
+                        >
+                            {[2022, 2023, 2024, 2025].map((year) => (
+                                <MenuItem key={year} value={year}>
+                                    {year}
+                                </MenuItem>
+                            ))}
+                        </Select>
                     </Box>
                     {/* Add a calendar to the right of the mainpage. */}
                     <Box
diff --git a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
index 0bb045a4..52ae6366 100644
--- a/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
+++ b/frontend/frontend/src/pages/subjectsPage/AddChangeSubjectPage.tsx
@@ -43,7 +43,8 @@ function UserList(
     loading: boolean,
     users: User[],
     setSelected: React.Dispatch<React.SetStateAction<number>>,
-    setOpen: React.Dispatch<React.SetStateAction<boolean>>
+    setOpen: React.Dispatch<React.SetStateAction<boolean>>,
+    handleRemove: () => void
 ) {
     return (
         <Box marginTop={3}>
@@ -70,7 +71,12 @@ function UserList(
                         {users.map((user) => {
                             const handleClickOpen = () => {
                                 setSelected(user.user)
-                                setOpen(true)
+
+                                if (user.is_lesgever) {
+                                    setOpen(true)
+                                } else {
+                                    handleRemove()
+                                }
                             }
                             {
                                 /* The list of users is mapped onto buttons
@@ -247,15 +253,12 @@ export function AddChangeSubjectPage() {
     }
 
     const handleRemoveStudent = (): void => {
-        setStudents((oldstudents: User[]): User[] => {
-            for (let i = 0; i < oldstudents.length; i++) {
-                if (oldstudents[i].user == selectedStudent) {
-                    oldstudents.splice(i, 1)
-                    return oldstudents
-                }
-            }
-            return oldstudents
-        })
+        const newstudents = students.filter(
+            (student) => student.user != selectedStudent
+        )
+
+        setStudents(newstudents)
+
         setOpenStudent(false)
     }
 
@@ -342,15 +345,10 @@ export function AddChangeSubjectPage() {
     // This function will remove a teacher from the list.
     // It does so by looping through the list and removing the teacher with the correct ID.
     const handleRemoveTeacher = () => {
-        setTeachers((oldteacher) => {
-            for (let i = 0; i < oldteacher.length; i++) {
-                if (oldteacher[i].user == selectedTeacher) {
-                    oldteacher.splice(i, 1)
-                    return oldteacher
-                }
-            }
-            return oldteacher
-        })
+        const newTeachers = teachers.filter(
+            (teacher) => teacher.user != selectedTeacher
+        )
+        setTeachers(newTeachers)
         setOpenTeacher(false)
     }
 
@@ -704,7 +702,8 @@ export function AddChangeSubjectPage() {
                                                             loading,
                                                             students,
                                                             setSelectedStudent,
-                                                            setOpenStudent
+                                                            setOpenStudent,
+                                                            handleRemoveStudent
                                                         )}
                                                     </Box>
                                                 </Card>
@@ -754,7 +753,8 @@ export function AddChangeSubjectPage() {
                                                             loading,
                                                             teachers,
                                                             setSelectedTeacher,
-                                                            setOpenTeacher
+                                                            setOpenTeacher,
+                                                            handleRemoveTeacher
                                                         )}
                                                     </Box>
                                                 </Card>

From 6b0f8cb1b508e5b0b46c3b8c7485ee2fc471648d Mon Sep 17 00:00:00 2001
From: Ben De Meurichy <ben.demeurichy@ugent.be>
Date: Mon, 20 May 2024 23:10:04 +0200
Subject: [PATCH 25/25] remove unused import

---
 .../src/components/RestrictionCard.tsx        | 32 ++++++++++---------
 1 file changed, 17 insertions(+), 15 deletions(-)

diff --git a/frontend/frontend/src/components/RestrictionCard.tsx b/frontend/frontend/src/components/RestrictionCard.tsx
index 75ad36ee..504238c6 100644
--- a/frontend/frontend/src/components/RestrictionCard.tsx
+++ b/frontend/frontend/src/components/RestrictionCard.tsx
@@ -1,8 +1,8 @@
 import { restriction } from '../pages/addChangeAssignmentPage/AddChangeAssignmentPage.tsx'
-import { Box, IconButton, Switch, Typography } from '@mui/material'
+import { IconButton, Switch, Typography } from '@mui/material'
 import CloseIcon from '@mui/icons-material/Close'
 import React from 'react'
-import {EvenlySpacedRow} from "./CustomComponents.tsx";
+import { EvenlySpacedRow } from './CustomComponents.tsx'
 
 interface RestrictionCardProps {
     restriction: restriction
@@ -29,19 +29,21 @@ export function RestrictionCard({
 
     return (
         <>
-            <EvenlySpacedRow items={[
-                <Typography id="script" variant={'body2'}>
-                    {restriction.script.replace(/^.*[\\/]/, '')}
-                </Typography>,
-                <Switch
-                    id="mustPassSwitch"
-                    value={mustPass}
-                    checked={mustPass}
-                    onChange={() => handleMustPassChange()}
-                />,
-                <IconButton id="closeButton" onClick={handleRemove}>
-                    <CloseIcon />
-                </IconButton>]}
+            <EvenlySpacedRow
+                items={[
+                    <Typography id="script" variant={'body2'}>
+                        {restriction.script.replace(/^.*[\\/]/, '')}
+                    </Typography>,
+                    <Switch
+                        id="mustPassSwitch"
+                        value={mustPass}
+                        checked={mustPass}
+                        onChange={() => handleMustPassChange()}
+                    />,
+                    <IconButton id="closeButton" onClick={handleRemove}>
+                        <CloseIcon />
+                    </IconButton>,
+                ]}
             />
         </>
     )