ADE-Scheduler icon indicating copy to clipboard operation
ADE-Scheduler copied to clipboard

Remove the project_id param

Open SnaKyEyeS opened this issue 4 years ago • 0 comments

Issue

Currently, the ADE API uses a project_id parameter to distinguish two academic years:

  • 8 for 2021-2022
  • 14 for 2020-2021 (if my memory serves me well)

This means that, based on the number provided in the request args (e.g. &code=FSA1BA&project_id=8), we get the events for the corresponding academic year. I believe this system is counter-intuitive as it requires changing project id's to view events from either year. Moreover, the automation process is annoying and a great source of errors/bugs whenever project_ids are added/or deleted (for example, this morning, the website was unusable because our server believed the project_id=14 was still valid even though it had been deleted).

Suggestion

I believe we, at ADE-Scheduler, in the spirit of making the interface the most intuitive and user-friendly possible, should remove the project_id entirely from the equation:

  • Remove all UI elements linked to project_id's
  • If several project_id's are present (essentially, this is the case during every summer, until the first day of the new academic year, around half September), merge all events together (e.g. remove the distinction &code=FSA1BA&project_id=8|14|...).
  • This could "easily" be done at the ADE API level, but requires changes for the remaining of the stack (from the schedule logic to the user interface) as the project_id param is present in a lot of functions.

SnaKyEyeS avatar Sep 14 '21 10:09 SnaKyEyeS