-
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
Feature/project routes #50
Conversation
not yet tested, might have some mistakes
Feature/savefiles
Feature/role interceptor
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, het enige wat ik nog zou aanpassen is om ApiRoutes
te gebruiken in plaats van bijvoorbeeld "/api/projects/"
te schrijven.
@usserwoutV2 zal ik doen |
* ApiRoutes no longer end on / * /courses/id returns a course and not a * list of projects (list of projects is moved to /courses/id/projects * /projects returns only projects user is part of * fixed recursive OneToMany relation between projects and deadline
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
This wil put dev up to date with main, aswell as adding the routes for projects and courses
the relevant issues are
closes #41
closes #54
closes #43
The relevant commits for the issues above are
commit project routes
commit course route