nusmods icon indicating copy to clipboard operation
nusmods copied to clipboard

Update faculty / department emails for Course Details error reporting

Open kokrui opened this issue 1 year ago • 1 comments

We have a "Report Errors" button on each Course Detail page. (See https://github.com/nusmodifications/nusmods/issues/3784)

However, some faculty emails listed there should be deprecated (e.g., GEQ1000, a course that no longer exists), and there are new emails which should be added.

We should incrementally improve this! If you notice data that should be added / removed / updated, please make a PR or issue to update https://github.com/nusmodifications/nusmods/blob/master/website/src/data/facultyEmail.json

kokrui avatar Aug 20 '24 11:08 kokrui

Hi, I'd like to work on this

dasha3412 avatar Aug 21 '24 07:08 dasha3412

Hi, wanted to ask about how we wanna go about doing this, do we wanna remove all the departments that no longer exist and all the emails for courses that no longer exist? Having a little trouble deciding which to remove in my PR.

I think eventually we'd want to remove all unused emails, but I'd be happy to accept PRs that make any sort of incremental progress (e.g. removing one or the other or any subset)!

kokrui avatar Feb 09 '25 01:02 kokrui

Sounds good, I'll start working on it!