Patch subject #240
Patch subject #240
Conversation
|
@reyniersbram er waren wat dingen gebroken door de veranderingen maar alles nu opgelost. om een rare reden is bijvoorbeeld een instructor die in de bestaande instructors zat nie identiek met dezelfde instructor die toegevoegd wordt. Hierdoor kreeg je dus problemen met de removedInstructors set wanneer je instructor meerdere malen verwijderd en weer toegevoegd. Daarom toch weer met uid's gewerkt daar. Voor de rest wat nullable problemen wat opgelost en ervoor gezorgt dat de instructor list niet update op het scherm terwijl de patch/post/delete requests bezig zijn |
|
oh help zoveel merge conflicten ik bekijk zo meteen |
# Conflicts: # frontend/src/components/subject/modify/header/ModifySubjectHeaderCard.vue # frontend/src/components/subject/subject/body/SubjectBody.vue # frontend/src/components/subject/subject/header/SubjectHeaderContainer.vue # frontend/src/components/subject/subjects/header/SubjectsHeaderContainer.vue # frontend/src/router/index.ts # frontend/src/views/subject/SubjectView.vue # frontend/src/views/subject/SubjectsView.vue
| @@ -15,7 +15,7 @@ | |||
| color="secondary" | |||
| variant="flat" | |||
| > | |||
| {{ `${academicYear}-${academicYear + 1}` }} | |||
| {{ `${academicYear - 2000}-${academicYear - 2000 + 1}` }} | |||
There was a problem hiding this comment.
Ik vindt het niet zo intuïtief dat de knop van het geselecteerde jaar donker is, en de andere knop wit.
reyniersbram
force-pushed
the
patch_subject
branch
from
db06778 to
f5087e3
Compare
* restructured files + first changes patch * passed down name and academic year props * null fix is in create * fixed router * title and academic year patch * added and deleted instructors * adding + deleting subject instructors * implementation patch subject mutation + error fix * fixes to form validation * adding and removing works * skeleton for query done * small academic year fix * slight cleanup * create works with patch * patch logic done * subjects refactoring modify * installed dialog component * i18n * lint format * implement patch mutation * fixes to broken patch page * fix broken merge conflicts * knoppen academie jaar subjects fix * lint format * fix van de fix van de academie jaar buttons * changed location of subject files in tests because of refactoring * add permission middleware --------- Co-authored-by: Bram Reyniers <[email protected]>
@reyniersbram
in de file PatchSubjectView staat de code om de uiteindelijke PATCH request te submitten in commentaar, met dus de queries.