Skip to content
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

Merged
merged 31 commits into from
May 20, 2024
Merged

Merge problemen oplossen #289

merged 31 commits into from
May 20, 2024

Conversation

sPAICEcake
Copy link
Contributor

PR om alle problemen die ontstaan zijn door de recentste merge op te lossen

@sPAICEcake sPAICEcake marked this pull request as draft May 20, 2024 13:51
@sPAICEcake sPAICEcake added bug Something isn't working frontend labels May 20, 2024
@sPAICEcake sPAICEcake added this to the milestone 3: afwerking milestone May 20, 2024
@sPAICEcake
Copy link
Contributor Author

indiening id's op assingmentpage kloppen ni meer

@sPAICEcake
Copy link
Contributor Author

deadline kalender knoppen verbeterde visuals verdwenen

@sPAICEcake
Copy link
Contributor Author

'Groep' knop op lesgever assignmentpage

@Bendemeurichy
Copy link
Contributor

Bendemeurichy commented May 20, 2024

Assignmentpage is volledig nagekeken, zou in orde moeten zijn.
zeg maar als er nog fouten opduiken

@gusvanpoucke
Copy link
Contributor

Deadline op assignmentpage is terug naar het oude formaat, oa toont de maand ipv het aantal minuten

@Bendemeurichy
Copy link
Contributor

back button addchange subject kapot

@gusvanpoucke
Copy link
Contributor

error bij het ophalen van groepsleden op assignmentpage

@gusvanpoucke
Copy link
Contributor

status van projecten word verkeerd getoond op de mainPage

@Bendemeurichy
Copy link
Contributor

status van projecten word verkeerd getoond op de mainPage

khad gezien in de code dat da idd op niks sloeg haha

@Bendemeurichy
Copy link
Contributor

Bendemeurichy commented May 20, 2024

image
toegevoegd bij de cancel button, het leek alsof save niks deed en dit was megaverwarrend

@gusvanpoucke
Copy link
Contributor

Het is mogelijk om een project zonder deadline aan te maken, maar dan verschijnt er overal Invalid Date of gewoon niks

@gusvanpoucke
Copy link
Contributor

addchangesubjectpage geeft geen waarschuwing als je een leerkracht aan studenten, of een student aan leerkrachten probeert toe te voegen maar doet gewoon niks.
De velden waar je de email invult worden ook niet gecleart

@sPAICEcake
Copy link
Contributor Author

sPAICEcake commented May 20, 2024

addchangesubjectpage geeft geen waarschuwing als je een leerkracht aan studenten, of een student aan leerkrachten probeert toe te voegen maar doet gewoon niks. De velden waar je de email invult worden ook niet gecleart

Dit heeft @Bendemeurichy hierboven net toegevoegd, enkel het clearen van velden moet nog

@gusvanpoucke
Copy link
Contributor

addchangesubjectpage geeft geen waarschuwing als je een leerkracht aan studenten, of een student aan leerkrachten probeert toe te voegen maar doet gewoon niks. De velden waar je de email invult worden ook niet gecleart

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,
hetzelfde verhaal voor een student die niet bestaat.

@sPAICEcake
Copy link
Contributor Author

addchangesubjectpage geeft geen waarschuwing als je een leerkracht aan studenten, of een student aan leerkrachten probeert toe te voegen maar doet gewoon niks. De velden waar je de email invult worden ook niet gecleart

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, hetzelfde verhaal voor een student die niet bestaat.

aah oei, ja kan wel idd

@gusvanpoucke
Copy link
Contributor

het is mogelijk om een vak zonder leerkrachten aan te maken
dit vak verschijnt dan in de databank, maar niemand kan eraan

@Bendemeurichy
Copy link
Contributor

het is mogelijk om een vak zonder leerkrachten aan te maken dit vak verschijnt dan in de databank, maar niemand kan eraan

ah ja klopt, ik heb enkel gezorgd datt je de laatste leerkracht niet kan verwijderen

@Bendemeurichy
Copy link
Contributor

je krijgt nu bij elke lesgever/ student dat je probeert te verwijderen uit een vak een popup. is dit niet te veel?

@sPAICEcake sPAICEcake marked this pull request as ready for review May 20, 2024 21:01
@sPAICEcake sPAICEcake requested a review from Bendemeurichy May 20, 2024 21:08
Copy link
Contributor

@gusvanpoucke gusvanpoucke left a 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

Copy link
Contributor

@Bendemeurichy Bendemeurichy left a 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

@sPAICEcake sPAICEcake merged commit 8999d02 into develop May 20, 2024
1 check passed
@sPAICEcake sPAICEcake deleted the merge_problems_fix branch May 20, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants