kukuminer / ta-app

Online application and selection platform for teaching assistants
0 stars 0 forks source link

Dev (#115) #116

Closed jonatanschroeder closed 2 months ago

jonatanschroeder commented 2 months ago

removed big unused code

so the user doesn't get navigated on server error (ie. invalid form data)

this is because it handles 403 by refreshing the page

only in student application

ErrorAlert is now not used anywhere, but it could be handy so I will leave it in as a component to maybe use later.

resolved by adding a tag that makes it so the parent node is a

instead of

MUI issue found here: https://github.com/mui/material-ui/issues/32289

  • fix nth-of-type warning

  • move files

  • remove comment

  • add first migration

  • Update index.js

  • Db migrations (#85)

  • migrations file runs

created and enabled migrations file. now to make it actually get migrations

  • now always runs all migrations

next to make it check if migrations are present. this means making a table to store present ones and checking length of table with length of files

  • migrations now run

if migrations table doesn't exist, it will display an error and run the migrations migration to enable migrations :)

  • Update runMigrations.js

  • Update runMigrations.js

  • modules working as intended

await migrations now properly awaits on top level

  • finish module migration

  • adjust db query

This means that the ID of the user must be in the instructor table to count as an instructor when importing sections

  • Update server/api/admin.js

  • Term wizard (#86)

  • term wizard page added

no functionality yet

  • prettify

  • create term endpoint

  • add table

  • table data populates properly

  • add default sort

  • Update datagrid_table.js

  • Add endpoint for updating term

  • Rename and add post feature

Currently has CRU out of CRUD, do we want delete capabilities? Or rename? I don't think it's important but maybe.

  • npm update (again server)

  • App push on apply (#97)

  • minor refactor + FE support

  • allow nav to be null

just in case it needs to be from a use source

  • backend properly prints latest courses

next to make it insert to DB

  • frontend properly posts

  • clicking "apply" now pulls app

NOTE: doesn't work if navigating to the term via URL!!!

  • formatting

  • todo

  • FE queries extra endpoint

  • abstract endpoint

also add filtering and everything should work now

  • remove on click

doubled effect, now runs when accessing via URL

  • fix 2 bugs

  • endpoint not sending response

  • FE sending null to endpoint security issue remains: if null is sent to endpoint, server might crash

  • 404 page now properly shows when term doesn't exist

  • remove unused var

  • redo func

  • npm install

  • npm updates

  • Endpoint error handler (#113)

  • add error catching

  • add package

  • add handler to index

  • add handler to endpoints

  • add handler to applicant

  • add handler to auth

  • Update instructor.js

  • change AS to its own import everywhere