Upgrade discord js from v12 to v13 - Githubissues

NYTDiscordBot / DiscordBot

0 stars 0 forks source link

Upgrade discord js from v12 to v13 #10

Closed DanceParty closed 2 months ago

DanceParty commented 2 months ago

Why are we upgrading? Trying to solve this issue. Found a related issue for a user on the same v12 version as us.

Changes accounted for:

Updated Jest, TS-Jest, node types,
Client must be passed options object containing at least a list of intents
On goml message handler, we are now manually checking the channel type
Migrated to the TextBasedChannel type instead of listing a random list of channel types
Updated node versions in workflows/test.yml since Discord.js doesn't support <v16 of node

Things to look out for

Potentially any missed changes (Guide here)
Potential type mis-match on TextBasedChannel
Requesting the incorrect or missing intents

What's next

After a bit of confirmation this is working in production, we should upgrade to v14 next (Guide)

DanceParty commented 2 months ago

Not sure how to remove the requirements for 10.x 12.x 14.x 15.x node test workflows 🤔