-
Notifications
You must be signed in to change notification settings - Fork 294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Discord.JS v14 #632
Conversation
Much appreciated. I'm merging this with my own fork and testing... |
The merged code seems to be working. One change that I can see is that the "SERVER MEMBERS INTENT Required for your bot to receive events listed under GUILD_MEMBERS." is now required whereas the old version didn't need it. Other than that, everything seems to have worked like a charm. |
Continuing to work on this project, I made a number of fixes/changes and also converted the application to typescript (which should not affect end-users). Changes that affect end-users:
Developer-relevant changes:
I made opinionated development changes and foolishly opened this PR against my main branch instead of a contribute branch, so these changes are also in this PR. I now understand this project is not actively maintained so I don't think it will be an issue. I attempted to remove the |
I updated the application.
Fixed some bugs/annoyances too:
Problems: