likuilin/course-explorer-monitor

Bot for the UIUC Course Watcher Discord server https://discord.gg/buwRCvm

course-explorer-monitor's Issues

Automatically display current semester somewhere

It'd be nice if the bot could automatically edit a message somewhere with the current semester every time it runs. This will make sure the "Current Semester" in #readme is always correct, especially if an older config file is accidentally deployed.

New meetings field in classes with ARRANGED (covid?) location

Instead of a meeting field, some sections have a meetings field with multiple meeting objects, and this causes JSON to be emitted to the end users:

ECE457: Section with CRN 74196 (MWF 11:00 AM - 11:50 AM) property meetings changed from {"meeting":[{"$":{"id":"0"},"type":[{"_":"Lecture","$":{"code":"LEC"}}],"start":["11:00 AM"],"end":["11:50 AM"],"daysOfTheWeek":["MWF "],"instructors":[{"instructor":[{"_":"Chen, X","$":{"lastName":"Chen","firstName":"X"}}]}]}]} to {"meeting":[{"$":{"id":"0"},"type":[{"_":"Lecture","$":{"code":"LEC"}}],"start":["11:00 AM"],"end":["11:50 AM"],"daysOfTheWeek":["MWF "],"roomNumber":["2015"],"buildingName":["Electrical & Computer Eng Bldg"],"instructors":[{"instructor":[{"_":"Chen, X","$":{"lastName":"Chen","firstName":"X"}}]}]}]}

It would be better if these were more intelligently parsed, for both diffing and display.

Refactor unwatch all code

Currently .unwatch all is implemented by a copy-paste of the .unwatch code, and DRY. This should ideally be abstracted out into a watch/unwatch function.

Permissions checks for commands

There isn't any permissions check for potentially disruptive commands like .restart and .forceupdate. Ideally, the admin (dev) role would be specified by ID in config and these commands should check for it.

If you're fixing this fixme, please make sure to also update the .watch DEV behavior, currently checking by hardcoded role name, in case users have a differently-named admin role.