2.0.0 #147
2.0.0 #147
Conversation
Removed the installer and all required packages. Removed the `loadCommand` and `unloadCommand` functions. Removed the `wait` function, was no longer used. Removed the `owners` functionality as the concept was too advanced for an example bot. Required the `logger` file where it was needed instead of binding it to `client`. Required the `settings` database and `config` file where needed. Moved the `clean` function into the `eval` command. Updated all of the code to reflect these changes. Moved the discord bot token into an `.env` file for better security. Added `owner` to the `.env` file. To-do; Change `levelCache` into a function to pull it off the `client` property.
whoops, missed one.
Removed `client` from `getSettings()` as it expects one variable, not two.
Found some stray `client.awaitReply` functions, updated the reload command to work with aliases.
| @YorkAARGH When is this going to be merged? |
| (not to sound impatient) 😄 |
When it's finished, there's a few things that remain to be done, then I'll need people to test it to eliminate the whole "It works on my machine" situation. |
Significantly reduced the `client` pollution by including a single `container` to the client and attaching every thing else, such as the `commands`, `aliases`, etc collections to that instead. Updated the code to reflect these changes.
Replaced the dead `moment`, and `moment-duration-format` dependencies with `@sapphire/time-utilities`. Replaced `chalk` with `colorette`.
Updated the timestamp for the logger to be consistent with previous versions.
Both `stat` commands included redundant parameters, they have been removed.
| Hey @YorkAARGH, I think we should keep the owner only function. Personally, I think that is something that people can learn from and dissect. |
I was initially in the same boat, but we don't cover teams at all in the guide and it wasn't really explained well, if at all in the code. The team functionality (to me at least) would imply using this example code in a large production based project, which we absolutely do not want to encourage as it lacks significant features that would be able to handle such a scale, which is why it was removed in the first place. |
* removed token lines from config.js * Update .gitignore
| Yay! 🎉 |
Please describe the changes this PR makes and why it should be merged, as well as any and all proof of successful tests:
Semantic versioning classification:
Removed the installer and all required packages.
Removed the
loadCommandandunloadCommandfunctions.Removed the
waitfunction, was no longer used.Removed the
ownersfunctionality as the concept was too advanced for an example bot.Moved the discord bot token into an
.envfile for better security.Added
ownerto the.envfile.Required the
loggerfile where it was needed instead of binding it toclient.Required the
settingsdatabase andconfigfile where needed.Created a
containerproperty to attachcommands,aliases,slashcmdsandlevelCacheto.Moved the
cleanfunction into theevalcommand.Replaced
chalkwithcolorette.Replaced
momentandmoment-duration-formatwith@sapphire/time-utilitiesUpdated all of the code to reflect these changes.