Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

update to discord.js v14 #349

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

update to discord.js v14 #349

wants to merge 1 commit into from

Conversation

mikejoh12
Copy link
Contributor

What issue is this solving?

Closes #348

Description

Updates the bot to use discord.js v14 which is using the Discord v10 api.

Any helpful knowledge/context for the reviewer?

Some of the changes required included modifying the code for building Embeds, adding GatewayIntentBits.MessageContent permission, modifying hex-colors to use 0x6096ba type format instead of '#6096ba' type format, and some error code/constant imports and naming.

I have tested all commands briefly but we should do some more testing of all commands including filter and deleted-msg-logging to make sure no new bugs are introduced.

  • Is a re-seeding of the database necessary? No
  • Any new dependencies to install? Yes, run npm install.

Please make sure you've attempted to meet the following coding standards

  • Code has been tested and does not produce errors
  • Code is readable and formatted
  • There isn't any unnecessary commented-out code

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bot is using Discord.js v13 when v14 is the latest stable release
1 participant