Class statistics: "Combined Excel" should match the other Excel downloads

UniversityOfHelsinkiCS / oodikone

An application for analyzing university data

https://oodikone.helsinki.fi

MIT License

14 stars 6 forks source link

Class statistics: "Combined Excel" should match the other Excel downloads #4551

Closed rikurauhala closed 3 months ago

rikurauhala commented 3 months ago

When using the "Combined Excel" button to download the contents of the General and Courses tables the results do not match what the individual Excel sheets have. There is some copy-pasted logic that has become outdated.

To fix, refactor and extract commonly used logic and make sure the titles and contents match the other two Excel files.

rikurauhala commented 3 months ago

We decided to remove the whole functionality instead.