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

redirecting to homepage when user not logged in #421

Merged
merged 2 commits into from
May 23, 2024

Conversation

AronBuzogany
Copy link
Contributor

Closes #398

@AronBuzogany AronBuzogany self-assigned this May 23, 2024
@Gerwoud Gerwoud self-requested a review May 23, 2024 18:44
Copy link
Contributor

@Gerwoud Gerwoud left a comment

Choose a reason for hiding this comment

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

🔥

Copy link
Contributor

@Vucis Vucis left a comment

Choose a reason for hiding this comment

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

You dropped this, King 👑

@Vucis Vucis merged commit 8b7aae1 into development May 23, 2024
2 checks passed
@Vucis Vucis deleted the frontend/fix/redirecting-when-not-logged-in branch May 23, 2024 18:46
Vucis added a commit that referenced this pull request May 23, 2024
* #15 - Using environment variable UPLOAD_FOLDER

* #15 - Add static typing to the submissions model

* #15 - Fixing linter

* docker-compose added

* #15 - Will fix this later

* #15 - Linter has cyclic imports

* change port

* #15 - Small URL change

* fixed extracting of zip and uploading in project upload directory

* fix
400 when non zip is uploaded

* fixed tests

* added test zip file

* linter fixes

* removed some test files

* linter and test fixes

* import depedency fix

* fix import order

* removed import getenv

* removed valid_project function

* fix fstring

* fix: upload_directory

* removed exist_ok=True

* use path.join

* added url field

* fixed not checking for tuple type anymore

* fixed env var for tests

* fixed env var for tests

* fixed with statements

* using os.path.split instead of regular split

* added exist_ok

* i forgot 💀 fix lmao yeet

* i forgot 💀 fix lmao yeet

* goofy augh fstring

* another small fix

* fixed the 'not fixed eh' problem

* linting

* fix handling fail

* added try block

* linter

* nginx.conf added

* rm conf

* fixed archieved to archived typo (#84)

* fixed archieved to archived typo

* fixed typo descriptions to description

* last change

* tests passed!

* #15 - Spelling

* #15 - Fixing tests the merge broke

* Feature/share link (#72)

* create subfolder for course tests

* added table for share codes

* loading env variables

* added default value to function parameters

* added endpoint for join_codes

* added tests for join_codes

* fixed: linting

* fixed typo:

* fixed import conftest

* fixed merge with singular model naming

* unused file

* added functionlity for downloading files

* reformatting

* linter

* niets veranderd? linter flipt

* small newline change

* removed f-string formatting

* added 404 and 500 cases

* fixed tests

* fixed os.curcwd

* linter

* removed print

* check if file exist and commented code removed

* fixed code duplication

* used basename insteal of .split

* linter fix

* Backend/tests/review models (#83)

* #81 - Updated user model tests, TODO delete

* #81 - Removing an unnecessary file

* #81 - Updated course model tests, TODO delete

* #81 - Updated course_relation model tests

* #81 - Cleanup

* #81 - Project model tests

* #81 - Submission model tests

* #81 - Cleanup

* #81 - Adding a test and also testing GH tests out

* #81 - Adding rollbacks after every error raised

* #81 - Maybe fix

* #81 - Another try

* #81 - This should fix it

* #81 Adding the delete method tests and updating the db_construct

* Tests/cleanup (#104)

* command useless and takes time

* cleaned up courses tests

* cleaned up project tests

* moved testzip to resources folder

* cleaned up users tests

* small cleanup submission tests

* resolved linting

* corrected function docs

* re-using existing fixtures to reduce code duplication

* resolved linter

* removed useless fstring

* removed unused import

* fix badge for workflow in readme.md (#108)

* test example workflow

* check if push argument works

* does the value update test

* Create ci-test-backend.yaml

created a seperate file for the backend test

* added backend badge to readme

* changed yml to yaml

* changed tests to test

* added seperate files for linter and tests for both front and backend

* gave correct names to workflows

* correct names to run name

* removed code dupe and pray for test run

* added both badges for linter and tests from backend

* file changes detected?

* removed unused code parts

* fix: eslint not found error

* added frontend badges

* frontend developper instructions (#110)

* frontend developper instructions

* only 1 backtick fix

* Merge backend authentication into development (#73)

* start authentication

* authentication start decorators

* login_required should work with access token

* backend authentication for most endpoints (very rough draft of functions in authentication.py)

* clean_up_function

* authentication cleanup

* give error when access_token fails

* documentation auth functions

* fixed imports

* actual import fix

* added requests

* authorize submissions

* removed double checks

* start testing setup backend authentication

* poging testen

* github tests check

* user tests with authentication

* auth url accessible hopefully

* change authorization to be easier to deal with since it doesn't matter for tests

* fixed jobCategory -> jobTitle

* fix authentication

* user tests zouden moeten slagen

* fix authentication arguments

* project tests with authentication

* changed auth server id of teacher

* maybe correct primary keys

* second try on primary key of course relations

* further test authentication

* authentication on project assignment files

* auth on course_join_codes and extra tests

* teacher_id in function when necessary

* user tests with authentication

* extra testing

* fixed comments

* lots of testing changes

* should be 1 error test now

* fix tests

* Fix #114 (#115)

* Fix #114

* test 1

* Fix #114 try 2

* fixed misnamed runner

* removed trailing new line

* Merge cleanup of authentication and tests into development (#111)

* start authentication

* authentication start decorators

* login_required should work with access token

* backend authentication for most endpoints (very rough draft of functions in authentication.py)

* clean_up_function

* authentication cleanup

* give error when access_token fails

* documentation auth functions

* fixed imports

* actual import fix

* added requests

* authorize submissions

* removed double checks

* start testing setup backend authentication

* poging testen

* github tests check

* user tests with authentication

* auth url accessible hopefully

* change authorization to be easier to deal with since it doesn't matter for tests

* fixed jobCategory -> jobTitle

* fix authentication

* user tests zouden moeten slagen

* fix authentication arguments

* project tests with authentication

* changed auth server id of teacher

* maybe correct primary keys

* second try on primary key of course relations

* further test authentication

* authentication on project assignment files

* auth on course_join_codes and extra tests

* teacher_id in function when necessary

* user tests with authentication

* extra testing

* fixed comments

* lots of testing changes

* should be 1 error test now

* fix tests

* small linter fix

* should fix the linter

* actually fixed this time

* linter all return statements

* allow admins to patch users

* should pass all tests

* mergeable

* grouped imports

* made requested changes

* requested changes

* line too long

* unused imports

* added models folder to utils

* Enhancement/debug env variable (#128)

* added waitress te requirements to serve in production

* added debug as an env variable and serving with waitress when not in debug mode

* fixed backend to frontend typo and added workflow branch argument (#119)

* fixed backend to frontend typo and added workflow branch argument

* added back 2. back to readme of backend

* changed submission status to enum (#133)

* changed submission status to enum

* resolved linting

* Fix #77 changed grading to float (#134)

* Fixed #112 (#113)

* Fixed #112

* removed new line

* Fix for uploading project files (#120)

* files are now patchable uncluding the files

* fix for uploading a file that is not a zip that it doesn't get saved

* unsaved changes

* buzoghany requested changes

* linter

* added rollbacks

* extra rollback

* Add get courses tests

* Add post courses tests

* Add get course tests

* Add patch course tests

* Add delete course tests

* Removes all usage of str(e) in courses endpoint (#126)

* Removed all usage of str(e) in courses endpoint

* common parts of error put into const var

---------

Co-authored-by: Siebe Vlietinck <[email protected]>

* changed port to 5001 (#142)

* Removed is_teacher and is_admin, changed to role enum (#127)

* Removed is_teacher and is_admin, changed to role enum

* sql enum type

* usage of enum and fixed sql

* made role serializable

* clean

* docs

* fixed user patch

* args vs json

* test

* check if course name is blank (#100)

* check if course name is blank

* generalized blank string check

* fixed yo request

* fixed conflicts

* cleaned code

* fix

* started initial readme for repo (#122)

* started initial readme for repo

* added workflow badges

* readme edits

* spelling mistakes

* grammar fix

* fixed route /docs and edited projects endpoint (#123)

* fixed route /docs and edited projects endpoint

* linter+test succeed

* fix: remove duplicate json file

* edited env variable for failing tests

* linter

* Base setup for easy translations in frontend (#145)

* i18n setup

* added dependencies

* Fix for issue 102. Submissions get now returns summary information instead of only url (#147)

* Fix #102

* linting

* Correctly uploading submission files (#136)

* 116 - Correctly uploading submission files

* #116 - Spelling

* Remove break

* Remove unused variable

* Specify which files are required

* Remove unnecessary stringify of UPLOAD_FOLDER

* Update the submission_path path.join

* Catching OSErrors when making directories and saving files

* Update the run_tests script

* This should allow frontend to communicate with our backend (#144)

* allow cors

* moved CORS to create_app_with_db

* Test invalid user form (#137)

* added test for invalid user form

* traling whitespace

* post to /users is not allowed

* removed double invalid fixture

* Test for course with invalid field (#125)

* added test for course with invalid field

* authorization header!

* fix

* Merge with backend/tests/auth

* courses get finished

* Feature/backend/documentation (#94)

* cooken on the projects documentation

* #92 - Fixing missing /submissions

* #92 - Fixing apilinter

* requested changes

* openapi object

* changed to yaml file

* small fix

* grammar

* json->yaml

* pyyaml installed in dev.txt

* linter

---------

Co-authored-by: Jarne Clauw <[email protected]>

* Base test class to handle auth (#153)

* Adding auth tokens

* Setup for auth tests

* Fixing tests

* Cleanup

* Spelling

* Remove unnecessary commit

* navbar (#80)

* navbar

* changes

* oops

* close bar on redirect

* linter conflicts for component function

* scalability matters

* useEffect to check login state and adjust link item list

* a little more readable

* No longer authenticating teachers/students through url params (#165)

* Fix #105

* adjusted tests to not use url query as authentication

* nested translation files (#163)

* return deadline aswel on projects (#168)

* Bump vite from 5.1.3 to 5.1.7 in /frontend (#166)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.1.3 to 5.1.7.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.1.7/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.1.7/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Using i18n browser detector and route detection to assign languages aswell as menu for users (#156)

* added tag to locales

* added menu to switch between languages

* updated i18n detector dependency

* configured detector to detect language in path

* configured language routes

* added outlet that detects languague and adds it to i18n

* added menu to select language

* resolved linting

* unused statement removed

* Frontend/feature/submission post (#150)

* added dependencies for treeview and code viewer

* created drag and drop upload, treeview and code viewer

* Fix #10

* removed unused dependencies

* created files utils with zip regex checker

* moved function to utils

* removed unused eslint rule

* corrected message on missing required files

* consistent frontend text

* fixed grammar

* resolved package diff

* Fixed inconsistently added auth tokens

* Whoops added it again

* Adding commented out tests

* Adding all the other auth tests

* Tests for students

* Admin tests

* Forgot part of merge

* Fix #179 (#180)

* Fix #179

* adjusted test to new endpoint

* fixed test

* fixed linting

* Whoops

* Hope this fixes it

* Typing and comments

* updating tests

* Set title of header using Portal (#177)

* Set title using Portal

* cleanup

* Removed paths from db for projects and allowing multiple deadlines (#185)

* Fix #181

* resolved syntax errors

* Fix #181

* removed unused statement

* resolved linting issues

* removed print statement

* Created dynamic deadline calender view/input (#190)

* Fix #188

* added mui date time components to dependencies

* fixed editable not being passed as prop

* Backend/feature/submission evaluation  (#67)

* added docker to requirements

* added: functionality to run a dockerized tester on a given submission

* added tests for submission evaluators

* moved file handling functions to seperate file

* added interpretor path

* updated dependencies

* fixed: linting

* added init file for module recognition

* CMD isn't used anymore as it is provided by the compose

* not running utils in docker as it uses docker internally and we don't yet have support for dind

* switched to LF

* changed: running tests not linting

* subfolders for submissions are nog longer made by the evaluator

* removed unused import statement

* linting: added blank line

* added ussage of requirements manifest which allows teachers to manifest which requirements are allowed to be used

* added: helper functions to prepare test cases

* added test case for working with requirements

* disabled pylinter for evaluator test resources

* fixed: linter ignoring all files

* using pylint recursive argument instead

* resolved merging issues

* using docker in docker image instead

* no longer binding socket to image

* passing path instead of project

* hotfix

* binding docker daemon

* starting dind docker daemon in entrypoint script

* removed unused env variable

* running all tests

* fixed linting

---------

Co-authored-by: Aron Buzogany <[email protected]>
Co-authored-by: abuzogan <[email protected]>

* Generalizing data field typing tests

* Cleanup + renaming

* Moving format to functions to make it clearer

* cleanup

* Fix test_post_admins

* Fix test_get_courses_wrong_parameter

* Creates project display for students and solves issue 186 (#187)

* init project display

* Fix #175

* added dependencies for #178

* Fix #186

* fixed test

* resolved linting

* fixed download url of submissions

* added title to header

* fixed typos

* merge fail

* Fixing all tests

* cleanup

* Generalizing some query parameter tests

* Fixing linter

* Whoops

* Making authorization tests more explicit

* registering composite types (#197)

* Added the runner used to test submissions as an optional field of project table (#199)

* Fix #198

* added possibility for custom runner

* fixed enum type name

* Running evaluator in synchron when runner is set (#203)

* Fix #200

* created executor

* fixed tests

* resolved linting

* linting

* fix wrong evaluator in test

* adjusted runner to support test

* Check if submission is LATE (#196)

* checking for if deadline is on time

* Fix #82

* removed redundant print statement

* authenticate endpoint

* setting submission status when no runner

* fixed inverted boolean

* router fix (#205)

* Added functionality to download submissions (#206)

* Fix #189

* fixed linter

* fixed linting

* fix the no status on workflow badges (#211)

* dev changes?

* removed branch argument

* removed random changes

* The submissions form for project page (#124)

* fixed beginning of project submission form

* changed

* changes

* changed

* removed print

* form works

* project form is working

* backend fixes

* backend linter

* linter

* fixed language support

* frontend linter

* packagelock

* linter

* letsgoo

* unused import :nerd:

* augh

* siebe review

* aron pr review

* merge conf

* pr fixed

* veel

* :)

* idk how it works but it does

* changes, a lot of them

* does this fix

* conftest

* dev changes?

* tests are working

* working regexes

* veel hihi

* removed unused React

* working form

* linter and tests

* small changes

* pr review jeej

* tag and route

* removed CORS

* removed unused imports

* hopefully aron is happy nowgit pushgit pushgit push!

* comment changed

* re added decorator (#214)

* Backend/project submission authorization issue (#210)

* Fixing authorization for project

* Fixing submission authorization issues

* cleanup

* Remove print and fix comments

* Fixing submissions

* Fixing projects

* Cleanup

* Fix filter placement

* error pages (#93)

* initial draft for error pages

* indentation fixed

* docs

* error pages v1

* indentation fixed

* layout adjusted, using RouterProvider instead of BrowserRouter to render errors correctly, tests added

* linter

* documentation

* linter error parameters

* Added i18n, changed general structure of App component and got rid of fade animation for the image

* linting, package-lock.json fix

* package file

* changes to paramters in ErrorPage.tsx

* use of keyPrefix

* keyPrefix

* fix

* Added endpoint to add a user using a join code (#218)

* Fix #71

* added course join endpoint

* internal server error is 500

* fixed code docs

* Addding a displayname to a user object (#221)

* Homepage student + landing page (#194)

Homepages added

* Full authentication (#216)

* backend-authentication

* fix projects

* remove extra requirements

* fix test-auth_server

* added: frontend redirect link

* fix backend auth

* fix error + code_challenge in frontend link

* line too long + frontend linter errors

* actual linter fix

* removed code_verifier until milestone 3

* added ability to log out and added everything to init

* redis added later

* added login button to header

* linter fix unused import

* add display_name to user creation

* test tests

* linter fix

* cookies test

* flask test cookies?

* added logout button

* with client

* Project overview (#219)

* goeie styling yallah

* pls werk

* werkt hopelijk

* linter

---------

Co-authored-by: Siebe Vlietinck <[email protected]>

* Project overview page (#222)

* start homepage

* added calender functions and titlecard

* homepage changes

* homepage changes

* Added: Homepage and Student Homepage

* homepage changes

* project projects parser

* test passed

* linter

* pr review

* Project card refactor

* homepage change fix

* Revert "Merge remote-tracking branch 'origin/backend/projectendpoint-fix' into frontend/feature/homepage"

This reverts commit 645443a, reversing
changes made to e5451b6.

* homepage changes

* rm comment

* pr changes

* project overview

* project overview

* project overview done

* pr changes

* project page changes

* added support for no deadline projects

* link changed

* link changed

* pr changes

* merged with HomeStudent.tsx

* projects overview page

* wording change

* homepage change

* homepage change

* API URL to API_HOST

* end point with /me

* project pages

* home

* create button

* create button

* opt

* deadline fix

* deadline fix

* deadline fix

* deadline fix

* creds

---------

Co-authored-by: gerwoud <[email protected]>

* Course views for teacher (#161)

* teacher detailed course base project list

* basic

* scrollers

* doc

* backtick strings

* base all courses page

* changed height

* first finished version of teacher course views

* teacher all course page reviewed

* display time until deadline

* more searchbars and translations

* changed file structure for maintainability

* titles and absolute create button

* searchbars update url and remain on reload

* user and admin listing

* reworked form and gave unique ids to all list items

* whoops linter ssst

* added .env to gitignore

* placeholder for loading courses

* debounce seachbars

* better debouncing more hooks

* setup for loader but doesnt work yet bc bad router

* loaders

* course detail teacher course join codes menu

* join code link points to app host now

* accident

* fixed empty filters, placeholders and teacher in admin list

* delete

* removed wannabe popup errorer

* unuserud

* package

* fix

* uhh

* u

* Fix authentication bugs (#226)

* fixed invalid_token_loader and added function to get csrf_cookie

* changed name of file to be correct

* small fixes

* added csrf to all fetches

* forgot an import

* formatting

* formatting

* added custom fetch function

* removed e

* Frontend/enhancement/join code endpoint (#248)

* Fix #247

* fixed linting

* using authenticated fetch

* unused import

* Fixes several buggs in join code endpoint (#251)

* Fix #249

* linting

* fix home/header login button (#228)

fix home/header login button #227 #241

* add label (#258)

* fix logout on server (#264)

* Fix #277 (#278)

* Fix #276 (#281)

* Changed AUTHENTICATION_URL .env variable (#268)

* changed authentication_url to test_authentication_url

* line too long

* Creates a general code evaluator (#256)

* Fix #255

* using apt to install node instead

* fixed using join instead of dirname

Co-authored-by: Jarne Clauw <[email protected]>

---------

Co-authored-by: Jarne Clauw <[email protected]>

* Authentication fix for backend tests (#280)

* fixed invalid_token_loader and added function to get csrf_cookie

* changed name of file to be correct

* small fixes

* added csrf to all fetches

* forgot an import

* formatting

* formatting

* added custom fetch function

* removed e

* some backend tests fixed hopefully

* added display names to test auth server

* set propagate exceptions to TRUE

* added checks in test auth

* csrf test

* get csrf token from response

* using cookie_jar

* return 401 on test auth when necessary

* get csrf token out of set-cookie headers

* fixed all user tests

* correct import this time

* actual correct import

* added default empty string so login does not crash

* fixed courses test

* fixed share link test

* fix project tests

* fixed submission tests

* change docker command

* removed authentication in no_auth test and correct authentication for project tests

* trying parameterized tests

* extra fixes

* actually correct fixes this time

* errors fixed

* fixed csrf error when wrong authentication

* Fix fill in the course selector (#261)

* fixed the issue

* make let const

* refactor api url

* refactor api url

* use effect

* use effect

* renamed for clarity

* use date not hours (#289)

* Cleanup of authentication tests (#285)

* Cleanup of authentication tests

* Adding tests where no csrf token is given

* Submissions download authorization (#293)

* Fixing

* Removing a test

* Courses now converts deadlines from string to Date right after fetch (#295)

* adjusted courses frontend to use deadlines Date instantiated right after fetch instead of str str 2d

* lintr

* removed TODO

* deadlines now have their own datatype!

* Submission authorization issues (#301)

* fix

* Fixing linter

* Submission issues (#304)

* Change the returned data from submission patch

* Updating the filtering on /submissions

* Return bad request on invalid keys in data

* Little fix

* Little fix

* fixing the f linter

* change path (#305)

* course path (#308)

* Navbar link to homepage added (#262)

* navbar link added

* rm i18

* Fix status on ProjectCards (#307)

* status fix

* rm space

* Fix #253 (#254)

* Seeder for easier demo and frontend testing (#291)

* basic seeder provider, course titles, usernames and random uid

* seeder creates and populates a few courses with random students where given uid is teacher

* ffix

* relocate, weird project deadline bug tho

* seeder go bvv

* fun with linter

* lint

* removed leading spaces

* batch operations and documentation cleanup

* bad env name

* more batch and better try except finally

* specific error

* close

* projects have 0-2 deadlines now, also randomized numprojects from fixed 2 to random 1-3

* 1-3

* import not good

* toml test

* toml

* faker is dev

* linter mad

* titles now in txt file

* completed toml info

* fixed parsing error

---------

Co-authored-by: Aron Buzogany <[email protected]>

* Redirecting when creation of project is succesful or showing error message on failure (#313)

* Fix #275

* removed unused import

* enhanced project submission error message in dutch

* When doing GET on courses, only returning courses you are authorized to see (#282)

* Fix #242

* fixed linting

* Added functionality for url queries

* fixed tests maybe

* changed error code on wrong parameter in tests to 500

* actually changed the assert this time

* courses currently ignores wrong filters

* ignore filter tests on courses for now

---------

Co-authored-by: Siebe Vlietinck <[email protected]>

* Fix #239 (#316)

* Fix #238 (#317)

* Fix #323 (#324)

* added paper islands, made project cards of equal size, displaying username instead of uid (#311)

* Added header on error pages and moved env import outside fetchme function (#319)

* added header to error pages

* moved env import outside function

* added check on posting project (#321)

* project form upload in user documentation (#250)

* project form upload

* added dutch translation

* PR review and styling

* siebe pr

* aron pr

* aron pr

* When no run_tests.sh is given when creating a project, cannot submit. (#326)

* trnanslation changes

* runner check

* changed everything to have 1 env var name (#332)

* Fix #274 (#314)

* added clipboard to copy the join code and redirecting to course when … (#325)

* added clipboard to copy the join code and redirecting to course when user already in course

* styling

* added dutch translation of python evaluator (#333)

* added dutch translation of python evaluator

* siebe pr

* Added readme for user guide (#330)

* added readme

* siebe pr

* Textfield performance issue (#334)

* added useTransition to wrap useState

* linter

* changed documentation location (#337)

* redirect lang fix (#339)

* add lang path (#341)

* uses display name instead of uid (#328)

* uses display name instead of uid

* re added authentication

* fix failing test

* changed APP var

* removed console log

* replaced path and merge conflicts

* removed 404 for submissions

* fixed page spamming backend with requests

* type error

---------

Co-authored-by: Aron Buzogany <[email protected]>

* Using react mui tabs in project view instead of implementing it again (#343)

* Fix #279

* linting

* Fix flickering with data loader (#348)

* added loader

* added loader

* linter

* fix name

* Seeder should generate unique submission times now (#354)

* unique deadlines

* unique submiss times

* max 10 days into past

* Project form/drag drop translations (#359)

* added some translations

* consistency!

* fixed axios.post to use authentication (#363)

* del (#365)

* outdated code (#358)

Co-authored-by: Warre Provoost <[email protected]>

* teacher added as student to course for real this time (#368)

* Submission tests (#362)

* Cleanup of authentication tests

* Adding tests where no csrf token is given

* Adding authentication tests

* Adding authorization tests

* Fixing most of the issues rn, but waiting on completion of other issue

* fix

* Fixing linter

* Fixing auth tests

* All tests written, but fixes needed

* Fixing most tests

* Fixing all tests except for the evaluator

* Trying to fix tests

* Fixing

* linter

* fix

* Ignore bad filters on fetches to endpoints (#360)

* test

* submissions now ignore bad filters

* small oopsies

* getatribute(Model, key, None), i forgot the None

* table__columns

* Teacher is now listed as admin on course page (#349)

* teacher is also listed, useMemo instead of useEffect to prevent listing twice

* useMemo bad, reworked the student and admin fetchers

* all fetches straight to loader fetches Me objects after fetching admins and students, no need for useffect

* fixed loader and seeder

* fix join codes bug (#374)

* Fix en-us bug (#366)

* fix en-us bug on homepage

* should be fixed everywhere now

* actually fixed hopefully

* fix bug req. change

* Fix the filter tests (#376)

* changed 200 to 400

* changed != to ==

* added filter functionlity to submissions

* removed prints

* linter

* linter again

* filters that aren't fields are now ignored in the code, tests for this are fixed

* yeet

* idk tbh

* Documentation/fix/GitHub link (#378)

* added table to frontend readme

* added .env vars to the README

* updated readme

* changed URL of api part

* changed github url

* changes not for this pr reverted

* newline goofy augh

* patch score in backend when edited in the frontend and i18n for page (#355)

* score patch is working

* changes

* added error message and i18n support

* added tooltip instead of alert :)

* linter

* update frontend readme variables (#356)

* added table to frontend readme

* added .env vars to the README

* updated readme

* changed URL of api part

* removed code verfier

* Frontend testing setup (#290)

* testing setup

* changed from jest to vitest and changed npm scripts to be easier to work with

* hopefully final version of config files

* fixed double import react bug

* setup finished

* linter fix

* forgot to push this

* any type

* requested changes

* vitest coverage generates coverage files

* fix JWT refresh to use database (#383)

* Update backend README (#357)

* added table to frontend readme

* added .env vars to the README

* added readme vars

* changes

* added explanation

* removed changes to frontend readme to exclude it from this pr

* remove code verifier

* added JWT application

* POST_HOST name

* added rfc link

* added microsoft admin link

* updated link

* fix requirements/docker dependency bug (#392)

* Fix #265 (#353)

* removed edit link (#396)

* added user guide links (#397)

* added user guide links

* added specific runner link

* small styling issue

* Add overview button to admin page of projects (#384)

* changes bekijken

* beaucoup

* edit title and description functionality

* linter :nerd:

* frontend linter

---------

Co-authored-by: Siebe Vlietinck <[email protected]>

* Fixed join code copy to clipboard (#404)

* fixed join code copy

* removed :

* Only return projects where u are student if visible for students is true (#403)

* projects visible for students

* lint

* added conditional rendering for edit funtionality (#402)

* added translation for homepage of user guide (#400)

* added translation

* fix

* Groups backend (#338)

* temp bad

* db_constr, model and first attempt at endpoint for group

* group prim key (group_id,project_id) added delete endpoint to leave groups, next up is test

* allow students in max 1 group

* model tests

* lint

* group menu frontend

* hm

* working endpoint for create and delete group

* translations

* begone front

* front removal

* lintr

* fixed changes, untested tho

* groups locked var should not mess up all older code

* only student or teacher can get groups ; unlock groups

* linter mad

* Very mad lintr

* vscode linter errors should be more obvi

* removed some teacher_id = None

* removed unused import

* bad prints

* unzipping submissions on submission (#406)

* unzipping submissions

* run_test -> run_tests

* run_test -> run_tests

* Fixed deadline time left (#407)

* fixed deadline time left

* > to >=

* Fix #408 (#409)

Co-authored-by: Warre Provoost <[email protected]>

* Filter on conditions the students projects (#411)

* projects visible for students

* lint

* filter on conditions

* lint

* fixed bad merge

* show red dots on home page (#412)

* Project issues (#410)

* Fixing the legacy warning

* Adding auth decorators to the project submission download enpoints

* Fixing query parameter tests

* remove bad param filter

* Project tests (#416)

* Auth tests

* Fixing the legacy warning

* Adding auth decorators to the project submission download enpoints

* Fixing auth tests

* Broken data field and query parameter tests

* Fixing query parameter tests

* More tests but first fixing some issues

* working tests

* remove bad param filter

* stuff

* added 403 for overview page (#414)

* added 403 for overview page

* linter

* added good response.ok placement and status code for 404

* linter

* Student course overview (#417)

* student overview

* linter

* linter

* linter

* check if teacher

* fixed authorizations in projectview (#418)

* fixed authorizations in projectview

* linting

* added locales

* redirecting to homepage when user not logged in (#421)

* redirecting to homepage when user not logged in

* linting

* only fetching admins when needed (#420)

* Fixed url flickering (#423)

* redirecting to homepage when user not logged in

* linting

* fixed url flickering

* fix seeder path (#422)

* added explanation (#425)

* added general evaluator documentation (#424)

* added general evaluator documentation

* fix 1

* fix 2

* fix yallah

* final fix i believe

* Header tests frontend (#391)

* file structure

* added hompage not logged in tests and header login functionality test

* linting + pass current tests hopefully

* more linting

* extra header tests

* other way of testing center button

* third option

* language test

* ignore screenshots

* logged in test

* formatting

* extra formatting

* deleted unused files

* intercept doesn't work on runner

* removed comments

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Jarne Clauw <[email protected]>
Co-authored-by: warre <[email protected]>
Co-authored-by: gerwoud <[email protected]>
Co-authored-by: Siebe Vlietinck <[email protected]>
Co-authored-by: Warre Provoost <[email protected]>
Co-authored-by: Gerwoud Van den Eynden <[email protected]>
Co-authored-by: Cedric Mekeirle <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Matisse Sulzer <[email protected]>
Co-authored-by: Siebe Vlietinck <[email protected]>
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.

Not a single page checks if the user is logged in
3 participants