Rewrite Meta-Issue #143
Description
While rewriting Winnie's codebase, we should be able to squash several bugs and resolve some of the outstanding issues.
Will complete as part of rewrite:
- Code Quality: Organise commands using a proper command/subcommand structure, and build a command parser that will properly handle this #66
- Code Quality: Replace the node-gameloop with a Redis-backed queue #73
- Code Quality: Implement Typescript #74
- Code Quality: Move database from MongoDB to PostgreSQL #76
- Code Quality: Update to discord.js v12 #98
- Feature Request: Set up a framework for translation and localisation #109
- Code Quality: Migrate infrastructure to Digital Ocean IaC with a managed database #144
Should be resolved as a side effect of rewrite:
- Bug: The auto-summary flag does not do anything. #57
- Bug: !stats command giving unknown error #89
- Bug: Goal raptor not granted despite meeting goal #92
- Bug: Deleting a Discord channel with an active war crashes the timer #94
- Bug: Wars in the middle of chains failing to start #100
Look at during rewrite - challenges:
- Code Quality: Make sprint an alias of war, and change the current sprint to race #88 (it's trivial, and now is the time to make this change, not later)
- Feature Request: Allow users to enter their starting word count at the beginning of a war #78 (this will eventually integrate with project tracking, but getting the basics in early is good)
- Bug: !list command not showing list of currently running wars #106 (we will probably have to go to embeds for the challenge list)
- Bug: Words per minute average at the end of a sprint is calculated incorrectly #107 (I fail at algorithmic maths? I expect this to be a really basic bug that we just need to test for)
- Bug: Wars can be cancelled after they have ended #110 (we should be able to just add a check for this, though there are notes on cancelling in the issue)
- Bug: Challenges can currently be cancelled even with users in other channels joined #115 (likewise to above)
- Feature Request: Server admins should be able to cancel challenges created in their servers #116 (same bucket as above two; will this be needed with some of the other changes?)
- Feature Request: Posting !join without a challenge ID will join you to the most recent challenge in the channel #117 (also potentially trivial, and again, now is the time)
Look at during rewrite - goals:
- Bug: Entering an invalid goal when attempting to reset discards the previous goal #95 (need to get the task sequence right, so validate before doing anything)
- Bug: Regular expression validates invalid timezone identifiers #105 (I have no idea how this ever validated; it could be a regex issue, and I think we're going to have to handle timezones differently anyways)
Look at during rewrite - other: