WekyDev / weky-npm

Other
72 stars 45 forks source link

Discord.js v13 Compatibility #40

Closed KingCh1ll closed 3 years ago

KingCh1ll commented 3 years ago

Hi! I created a github repo to support Discord.js v13 for weky.

GitHub Repo Link https://github.com/kingch1ll/weky-npm-v13

KingCh1ll commented 3 years ago

The module, Discord-buttons doesn't work with Discord.js v13 and is pointless at that (since Discord v13 supports it). I removed it and replaced it with Discord v13.

RaZeSloth commented 3 years ago

Why not fork this repo and make a PR? It is kinda confusing on why make a whole new repo

sujalgoel commented 3 years ago

Why not fork this repo and make a PR? It is kinda confusing on why make a whole new repo

yep

KingCh1ll commented 3 years ago

Why not fork this repo and make a PR? It is kinda confusing on why make a whole new repo

I tried to do so. It gave me an odd error.

sujalgoel commented 3 years ago

Why not fork this repo and make a PR? It is kinda confusing on why make a whole new repo

I tried to do so. It gave me an odd error.

what is that error?

KingCh1ll commented 3 years ago

Why not fork this repo and make a PR? It is kinda confusing on why make a whole new repo

I tried to do so. It gave me an odd error.

what is that error?

I don't really remember to be honest. It's been almost 4 days after all.

rayzdev commented 3 years ago

I took a look at the Calculator command on your repo. It's not working with v13. You are still using .setID instead of .setCustomId.

rayzdev commented 3 years ago

Yeah um you basically just removed discord-buttons and added MessageButton and MessageActionRow imports for discord.js. This won't work with discord.js v13.