Remove remindme functionality

[MattyTheHacker]

As discussed on the CSS Discord server (and approved by committee), /remind-me functionality is covered by other existing bots and so can be removed.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[CarrotManMatt]

The beauty of separate cogs where whole functionality can be removed so quickly!

[CarrotManMatt]

Sorry I approved without thinking. Can the DB functionality be soft-deprecated in some way? So that the models still exist and the DB doesn't become corrupted, but the functionality cannot be used or new data cannot be created. The DiscordReminder model is the problem here.

[MattyTheHacker]

Sorry I approved without thinking. Can the DB functionality be soft-deprecated in some way? So that the models still exist and the DB doesn't become corrupted, but the functionality cannot be used or new data cannot be created. The DiscordReminder model is the problem here.

I mean I suppose I could try but is it worth it? it'd leave a mess in the codebase for something that realistically wouldn't give anyone any benefit?

I actually forgot about the db models, so in theory that's the situation we have now right? the models still exist but the ability to modify them would be gone? This does remind me I do also need to remove the delete-all-reminders command too though so I'll do that now.

[CarrotManMatt]

Ah I think the best bet is to just mark the DB class with @warnings.deprecated.

https://typing.python.org/en/latest/spec/directives.html#deprecated

[MattyTheHacker]

Ah I think the best bet is to just mark the DB class with @warnings.deprecated.

https://typing.python.org/en/latest/spec/directives.html#deprecated

Which class? the commands cog?

[CarrotManMatt]

Sorry, db.core.models.DiscordReminder DB class. It can also be removed from __all__ in that file

[CarrotManMatt]

I suggest a note is added to the corresponding element (delete_all_reminders) in TeXBotBaseCog.ERROR_ACTIVITIES to mention that the command has been removed. But still leave the message existing in the dictionary just in case that key appears during an exception handling for some wild reason.

TeX-Bot-Py-V2/utils/tex_bot_base_cog.py

Lines 38 to 40 in 8d28d61

	"delete_all_reminders": (
	"delete all `DiscordReminder` objects from the backend database"
	),

[MattyTheHacker]

I suggest a note is added to the corresponding element (delete_all_reminders) in TeXBotBaseCog.ERROR_ACTIVITIES to mention that the command has been removed. But still leave the message existing in the dictionary just in case that key appears during an exception handling for some wild reason.

TeX-Bot-Py-V2/utils/tex_bot_base_cog.py

Lines 38 to 40 in 8d28d61

	"delete_all_reminders": (
	"delete all `DiscordReminder` objects from the backend database"
	),

done