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

Add skeleton code #1

Merged
merged 8 commits into from
Feb 23, 2024
Merged

Add skeleton code #1

merged 8 commits into from
Feb 23, 2024

Conversation

xerbalind
Copy link
Contributor

No description provided.

@xerbalind
Copy link
Contributor Author

pull request test ook de github actions

Copy link
Contributor

@reyniersbram reyniersbram left a comment

Choose a reason for hiding this comment

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

Top! Handig dat dit al gedaan is. Steken we misschien in de backend app.py niet ook in een src directory zodat dat ook allemaal mooi gesplitst blijft?

@xerbalind
Copy link
Contributor Author

code zou ik ook in src directory doen, maar app.py is zowat de entrypoint om backend te starten dus kzou dat gelijk in root backend directory steken

@reyniersbram
Copy link
Contributor

Ok klinkt goed.

reyniersbram
reyniersbram previously approved these changes Feb 20, 2024
@reyniersbram
Copy link
Contributor

reyniersbram commented Feb 20, 2024

Ik heb net nog eens de skeleton code van de frontend opnieuw bekeken en zie dat je precies niet voor de 'Single Page Application' optie bent gegaan. Is daar een reden voor?
En testen met Vitest ontbreekt ook zie ik net.

@reyniersbram reyniersbram dismissed their stale review February 20, 2024 18:48

Toch nog enkele ontbrekingen gevonden

@xerbalind
Copy link
Contributor Author

xerbalind commented Feb 20, 2024

euh ja kheb gwn nee gedaan op alle extra dingen, met de bedoeling dat dit later nog toegevoegd kan worden (voor de dingen die nodig zijn)

@reyniersbram reyniersbram merged commit 575f21c into main Feb 23, 2024
1 check passed
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.

2 participants