-
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
Release 3 #258
Merged
Merged
Release 3 #258
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* testen proberen fixen * admin frontend test * format
* quick commit * quick commit * quick commit * almost finished * finished + linter :) * linter again :'( * linter again x2 :'( * groupsview fixes * groupview fixes * fix 'join group' button url * fix type imports * run formatter * small fix * linter :) --------- Co-authored-by: Bram Reyniers <[email protected]>
* save surname of user in database #144 * help automatic migration * run formatter * use surname on homescreen * run formatter * remove print statement * Update backend/src/auth/router.py Co-authored-by: Xander Bil <[email protected]> --------- Co-authored-by: Xander Bil <[email protected]>
* style * style * temp * style * style * temp * more styling * testen * lint * format * styling * some styling en test fixes * backbuttons * subjects styling * subject styling * create project styling * create subjects styling * lint * format * format * requested changes * format * fix show group * more requested changes * small groups fix --------- Co-authored-by: Bram Reyniers <[email protected]>
* generic permission middleware * placeholder middleware implementations * implement isAdmin check * permissions on admin page + fix typo * logical functions for conditions * pass MiddlewareContext to conditions * permission to check if user is part of subject * permission to check if user can create a project for a subject * run formatter * restructure navigation guard middlewares * subject details permissions * project details permissions * go to /not-found instead of /forbidden * create project permissions * fix project permission checks * permissions for submit page * permissions for groups overview * permissions for group details * permissions for submission list of project * remove unused imports * Apply suggestions from code review Co-authored-by: Xander Bil <[email protected]> * permissions for new pages * allow admins to visit all pages --------- Co-authored-by: Xander Bil <[email protected]>
* styling * styling * styling * styling * submissionslist test * meer testen * format * merge conflicts * lint + format * button verplaatsen * styling klaar * testen * fix test after dev merge * projectview test * more tests * SubmissionsTeacherView.spec.ts * button change position * format * change submissioncard * format
This reverts commit e84929e.
* fix: invalid redirect after create project * frontend fix: joining group when automatically now works * fix: register to subject both as student and instructor * fix submect mail is null * fix #185 * fix weird groups styling * fix broken router push * fix inconsistent academic year * fix: duplicates in files * format * fix year * fix test? * fix group id showing instead of num --------- Co-authored-by: Bram Reyniers <[email protected]>
* restructured files + first changes patch * passed down name and academic year props * null fix is in create * fixed router * title and academic year patch * added and deleted instructors * adding + deleting subject instructors * implementation patch subject mutation + error fix * fixes to form validation * adding and removing works * skeleton for query done * small academic year fix * slight cleanup * create works with patch * patch logic done * subjects refactoring modify * installed dialog component * i18n * lint format * implement patch mutation * fixes to broken patch page * fix broken merge conflicts * knoppen academie jaar subjects fix * lint format * fix van de fix van de academie jaar buttons * changed location of subject files in tests because of refactoring * add permission middleware --------- Co-authored-by: Bram Reyniers <[email protected]>
* finishing touches * requested change
Bumps [requests](https://github.com/psf/requests) from 2.31.0 to 2.32.2. - [Release notes](https://github.com/psf/requests/releases) - [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md) - [Commits](psf/requests@v2.31.0...v2.32.2) --- updated-dependencies: - dependency-name: requests dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ty == 1 (#248) * finishing touches * requested change * fix reroutes submissionspage
* fixes remove user cascading on groups/subjects/... * linter
* ugent logo * logo on loginscreen * default lang
create files to use in demo
* fix group wrong num error * dries fix --------- Co-authored-by: Xander Bil <[email protected]>
fix issue #234 (add publish date checks)
whoop 🎉 |
littt 🔥 |
hypeeee 🥳 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.