-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge problemen oplossen #289
Conversation
indiening id's op assingmentpage kloppen ni meer |
deadline kalender knoppen verbeterde visuals verdwenen |
'Groep' knop op lesgever assignmentpage |
Assignmentpage is volledig nagekeken, zou in orde moeten zijn. |
Deadline op assignmentpage is terug naar het oude formaat, oa toont de maand ipv het aantal minuten |
back button addchange subject kapot |
error bij het ophalen van groepsleden op assignmentpage |
status van projecten word verkeerd getoond op de mainPage |
khad gezien in de code dat da idd op niks sloeg haha |
…dent dat die nog niet in een groepje zit
… into merge_problems_fix
Het is mogelijk om een project zonder deadline aan te maken, maar dan verschijnt er overal Invalid Date of gewoon niks |
addchangesubjectpage geeft geen waarschuwing als je een leerkracht aan studenten, of een student aan leerkrachten probeert toe te voegen maar doet gewoon niks. |
Dit heeft @Bendemeurichy hierboven net toegevoegd, enkel het clearen van velden moet nog |
Er verschijnt een waarschuwing als je het probeert op te slaan maar nog niet als je de student toe probeert te voegen denk ik, |
aah oei, ja kan wel idd |
het is mogelijk om een vak zonder leerkrachten aan te maken |
ah ja klopt, ik heb enkel gezorgd datt je de laatste leerkracht niet kan verwijderen |
je krijgt nu bij elke lesgever/ student dat je probeert te verwijderen uit een vak een popup. is dit niet te veel? |
… into merge_problems_fix
cleanup restrictions box in addchangeAssigmentPage
…gebruikers probeert toe te voegen, en de input velden worden automatisch gecleart
… into merge_problems_fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ziet er allemaal goed uit!
Hiermee zouden de laatste bugs er allemaal uit moeten zijn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ziet er goed uit, fijn dat we nog zo veel hebben kunnen verbeteren
PR om alle problemen die ontstaan zijn door de recentste merge op te lossen