chore: PR comments

SELab-2 · May 21, 2024 · 95698b3 · 95698b3
1 parent 9a34516
commit 95698b3
Show file tree

Hide file tree

Showing 9 changed files with 26 additions and 12 deletions.
diff --git a/backend/api/serializers/checks_serializer.py b/backend/api/serializers/checks_serializer.py
@@ -99,7 +99,7 @@ def validate(self, attrs: dict) -> dict:
         data = super().validate(attrs)
 
         # Check if the docker image is provided
-        if not "docker_image" in self.initial_data:
+        if "docker_image" not in self.initial_data:
             raise serializers.ValidationError(_("extra_check.error.docker_image"))
 
         # Check if the docker image exists

diff --git a/backend/api/tests/test_file_structure.py b/backend/api/tests/test_file_structure.py
@@ -49,8 +49,6 @@ def test_parsing(self):
 
         content_json = json.loads(response.content.decode("utf-8"))
 
-        print(project, content_json)
-
         self.assertEqual(len(content_json), 6)
 
         expected_project_url = settings.TESTING_BASE_LINK + reverse(

diff --git a/backend/api/views/project_view.py b/backend/api/views/project_view.py
@@ -20,6 +20,7 @@
 from rest_framework.response import Response
 from rest_framework.viewsets import GenericViewSet
 
+
 # TODO: Error message when creating a project with wrongly formatted date looks a bit weird
 class ProjectViewSet(RetrieveModelMixin,
                      UpdateModelMixin,

diff --git a/frontend/src/assets/lang/app/en.json b/frontend/src/assets/lang/app/en.json
@@ -71,7 +71,12 @@
             "noStudents": "No students in this group",
             "locked": "Closed",
             "unlocked": "Open",
-            "structureChecks": "Submission structure",
+            "structureChecks": {
+                "title": "Structure checks",
+                "placeholder": "Give a name to this folder",
+                "cancelSelection": "Deselect {0}",
+                "newFolder": "New folder"
+            },
             "extraChecks": {
                 "title": "Automatic checks on a submission",
                 "empty": "No checks addeed",

diff --git a/frontend/src/assets/lang/app/nl.json b/frontend/src/assets/lang/app/nl.json
@@ -60,7 +60,6 @@
             "create": "Creëer nieuw project",
             "save": "Project opslaan",
             "edit": "Project bewerken",
-            "structureChecks": "Indieningsstructuur",
             "name": "Projectnaam",
             "description": "Beschrijving",
             "startDate": "Start project",
@@ -73,6 +72,12 @@
             "noStudents": "Geen studenten in deze groep",
             "locked": "Gesloten",
             "unlocked": "Open",
+            "structureChecks": {
+                "title": "Indieningsstructuur",
+                "placeholder": "Geef deze nieuwe map een naam",
+                "cancelSelection": "Deselecteer {0}",
+                "newFolder": "Nieuwe map"
+            },
             "extraChecks": {
                 "title": "Automatische checks op een indiening",
                 "add": "Nieuwe check",

diff --git a/frontend/src/components/projects/ExtraChecksUpload.vue b/frontend/src/components/projects/ExtraChecksUpload.vue
@@ -125,6 +125,7 @@ async function onDockerImageUpload(event: any): Promise<void> {
         :icon="PrimeIcons.PLUS"
         :label="t('views.projects.extraChecks.add')"
         icon-pos="left"
+        rounded
     />
 
     <!-- Dialog with a form to create a new extra check -->

diff --git a/frontend/src/components/projects/ProjectForm.vue b/frontend/src/components/projects/ProjectForm.vue
@@ -202,7 +202,7 @@ watchEffect(() => {
                 <!-- Define the submission structure checks -->
                 <div class="grid">
                     <div class="field col">
-                        <label for="structure">{{ t('views.projects.structureChecks') }}</label>
+                        <label for="structure">{{ t('views.projects.structureChecks.title') }}</label>
                         <ProjectStructureTree id="structure" v-model="form.structure_checks" />
                     </div>
                 </div>

diff --git a/frontend/src/components/projects/ProjectStructureTree.vue b/frontend/src/components/projects/ProjectStructureTree.vue
@@ -7,10 +7,14 @@ import { StructureCheck } from '@/types/StructureCheck.ts';
 import { computed, ref } from 'vue';
 import { type TreeNode } from 'primevue/treenode';
 import { PrimeIcons } from 'primevue/api';
+import { useI18n } from 'vue-i18n';
 
 /* Models */
 const structureChecks = defineModel<StructureCheck[]>();
 
+/* Composables injections */
+const { t } = useI18n();
+
 /* State */
 const selectedStructureCheck = ref<StructureCheck | null>(null);
 const editingStructureCheck = ref<StructureCheck | null>(null);
@@ -99,7 +103,7 @@ function addStructureCheck(check: StructureCheck | null = null): void {
             hierarchy = selectedStructureCheck.value.getDirectoryHierarchy();
         }
 
-        hierarchy.push('new');
+        hierarchy.push(t('views.projects.structureChecks.placeholder'));
 
         structureChecks.value.push((editingStructureCheck.value = new StructureCheck('', hierarchy.join('/'))));
     }
@@ -171,7 +175,7 @@ function newTreeNode(check: StructureCheck, key: string, label: string, leaf: bo
                         class="w-full"
                         :model-value="node.data.getObligatedExtensionList()"
                         @update:model-value="node.data.setObligatedExtensionList($event)"
-                        v-tooltip="'Verplichte extensies'"
+                        v-tooltip="t('views.projects.structureChecks.obligatedExtensions')"
                     >
                         <template #chip="{ value }">
                             {{ value }}
@@ -183,7 +187,7 @@ function newTreeNode(check: StructureCheck, key: string, label: string, leaf: bo
                         class="w-full"
                         :model-value="node.data.getBlockedExtensionList()"
                         @update:model-value="node.data.setBlockedExtensionList($event)"
-                        v-tooltip="'Verboden extensies'"
+                        v-tooltip="t('views.projects.structureChecks.blockedExtensions')"
                     >
                         <template #chip="{ value }">
                             {{ value }}
@@ -227,15 +231,15 @@ function newTreeNode(check: StructureCheck, key: string, label: string, leaf: bo
                 :icon="PrimeIcons.TIMES"
                 v-if="selectedStructureCheck !== null"
                 severity="contrast"
-                :label="'Deselecteer ' + selectedStructureCheck.path"
+                :label="t('views.projects.structureChecks.cancelSelection', [selectedStructureCheck.path])"
                 @click="
                     selectedStructureCheck = null;
                     editingStructureCheck = null;
                 "
                 outlined
                 rounded
             />
-            <Button label="Nieuwe map" :icon="PrimeIcons.FOLDER" @click="addStructureCheck()" outlined rounded link />
+            <Button :label="t('views.projects.structureChecks.newFolder')" :icon="PrimeIcons.FOLDER" @click="addStructureCheck()" outlined rounded />
         </div>
     </div>
 </template>

diff --git a/frontend/src/views/authentication/LoginView.vue b/frontend/src/views/authentication/LoginView.vue
@@ -53,7 +53,7 @@ const { t } = useI18n();
     </BaseLayout>
 </template>
 
-<style lang="scss">
+<style lang="scss" scoped>
 #wrapper {
     min-height: calc(100vh - (150px + 173px + 7rem)) !important;
 }