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

Frontend visuals done #285

Merged
merged 28 commits into from
May 19, 2024
Merged

Frontend visuals done #285

merged 28 commits into from
May 19, 2024

Conversation

friedrecursion
Copy link
Contributor

No description provided.

@friedrecursion friedrecursion requested review from Bendemeurichy and gusvanpoucke and removed request for Bendemeurichy and gusvanpoucke May 19, 2024 18:53
@friedrecursion friedrecursion requested review from ArnoutAllaert, mathis2003 and sPAICEcake and removed request for ArnoutAllaert May 19, 2024 19:09
@mathis2003
Copy link
Contributor

De code ziet er goed uit, maar ik heb wel dit gekregen toen ik het begon te runnen:
Schermafbeelding 2024-05-19 om 21 29 45
Ik zal het eens proberen fixen en als het lukt pushen. Ik vind het wel raar dat het bij u dan wel werkte, misschien zijn onze instellingen m.b.t. imports anders ofzo.

@mathis2003
Copy link
Contributor

mathis2003 commented May 19, 2024

Ok deze commit heeft het probleem verholpen. Maar tijdens het testen van de pagina krijg ik wel nog een rare error.
Als ik naar de infosec pagina ga en op het Online voting project klik, krijg ik de volgende error message te zien:

Something went wrong.
relative_group_id is not defined

"Groepen test" heeft dit probleem ook.
Het rare is dat "Test InfoSec Project" dit probleem niet heeft.

@mathis2003
Copy link
Contributor

Tweede bug: De toevoegknop op de mainpage van de lesgever werkt ook niet. We kunnen dat eventueel achteraf nog oplossen. Maar het is misschien best om dat al gedaan te hebben als het niet teveel werk is.

@friedrecursion
Copy link
Contributor Author

Tweede bug: De toevoegknop op de mainpage van de lesgever werkt ook niet. We kunnen dat eventueel achteraf nog oplossen. Maar het is misschien best om dat al gedaan te hebben als het niet teveel werk is.

kzal da nu nog proberen fixen

@mathis2003
Copy link
Contributor

Derde bug: de potlood knop bij de vakpagina, dus om een vak aan te passen, geeft ook een error:

Something went wrong.
Button is not defined

Maar dat is misschien ook een zorg voor later.

@friedrecursion
Copy link
Contributor Author

Deze bugs werden al verholpen op de frontend branch

Copy link
Contributor

@mathis2003 mathis2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, voor de rest ziet het er heel mooi uit!

@friedrecursion friedrecursion removed the request for review from sPAICEcake May 19, 2024 19:47
@friedrecursion friedrecursion merged commit fa610ed into frontend May 19, 2024
@sPAICEcake sPAICEcake mentioned this pull request May 20, 2024
@friedrecursion friedrecursion deleted the frontend_visuals branch May 21, 2024 12:58
@friedrecursion friedrecursion restored the frontend_visuals branch May 21, 2024 12:58
@sPAICEcake sPAICEcake deleted the frontend_visuals branch May 23, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants