UQComputingSociety / uqcsbot-discord

:mortar_board: UQCSbot: Our friendly little Discord bot
https://discord.uqcs.org
MIT License
20 stars 20 forks source link

Remove Ununimplemented #168

Closed bradleysigma closed 1 year ago

bradleysigma commented 1 year ago

Went to reimplement something from the Slackbot, only to find about the third of the so called unimplemented commands were no longer so.

bradleysigma commented 1 year ago

Also removed cookbook.py, as it's implemented by /repo find.

JamesDearlove commented 1 year ago

I'm actually of the opinion that we should remove the entire unimplemented folder, as was previously discussed internally.

Now the bot is pretty much running entirely on the newer application command system, creating new commands looks a bit different to the Slack bot and any remaining useful commands might benefit from a rewrite to support Discord's unique features. Any commands that might be useful can be migrated to issues and linked to this repos history or directly to the Slack bot history

bradleysigma commented 1 year ago

FWIW, after the Slackbot migrated from JavaScript to Python, there were a whole bunch of commands that weren't initially transferred, and similarly chucked into an unimplemented folder. An issue was made for these commands as a whole, but after a year or so, the last eleven commands had their own individual issues created.

bradleysigma commented 1 year ago

Also, the remaining non-Discord/Slack Python code is of variable usefulness; the URL that ascii.py uses, for example, no longer exists.