Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove Reminder Cog #130

Open
rtk-rnjn opened this issue Jun 7, 2023 · 1 comment
Open

remove Reminder Cog #130

rtk-rnjn opened this issue Jun 7, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request priority:medium

Comments

@rtk-rnjn
Copy link
Contributor

rtk-rnjn commented Jun 7, 2023

The Reminder Cog is a cog that does not have any commands or events.

I propose that we completely remove the Reminder Cog and transfer all of its functions and methods to the bot core. This will simplify the codebase and make it easier to maintain.


Benefits

  • Simplification of the codebase
  • Easier maintenance
  • Reduced risk of bugs

Risks

  • Potential for downtime

I believe that the benefits of removing the Reminder Cog outweigh the risks.

@deadaf
Copy link
Member

deadaf commented Jun 11, 2023

yes this actually makes sense, a lot of functionalities depends on the reminder cog at the moment and if for any reason this cog isn't loaded, a lot of things will break.
I will look into it.

@deadaf deadaf self-assigned this Jun 11, 2023
@deadaf deadaf added enhancement New feature or request priority:medium labels Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority:medium
Projects
None yet
Development

No branches or pull requests

2 participants