Skip to content
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 master branch to guidebot v13 #144

Merged
merged 9 commits into from
Aug 8, 2021
Merged

Update master branch to guidebot v13 #144

merged 9 commits into from
Aug 8, 2021

Conversation

YorkAARGH
Copy link
Member

Not sure what to put here other than updating the master branch to the latest code.

YorkAARGH and others added 8 commits July 7, 2021 22:39
Updated everything
updated from `user.id` to `ownerId`
* Adding partials to update baseConfig

* Adding logical "restart" alias
Updated all the commands that output codeblocks, used the new formatters.
Added a single line of code to make Guidebot auto-join any threads created, useful for moderation bots.
@YorkAARGH YorkAARGH added enhancement guidebot The original version of guidebot. non-issue Not an issue, asking a question or making a statement. labels Aug 8, 2021
@YorkAARGH
Copy link
Member Author

YorkAARGH commented Aug 8, 2021

guild.owner was removed in v13, use guild.ownerId or guild.fetchOwner to get the ID.

discordjs/discord.js#5480

I only just noticed this comment, but regarding what you said about guildCreate, if you actually look at the code it clearly uses guild.ownerId

e716d4c#diff-fb77c4ccc7591d343e96544ec63ab6009a876f338e46924d5cbf868edeaccb4d (from this commit)

* Slash Commands

Added slash command support to the original (main) version of guidebot.

Updated discord.js to version 13.0.1, updated the throw in the index to require version 16 of nodejs.

Brought both versions of guidebot to the same functional level.

* Fixed a small mistake

The setup placed the partial array inside an array causing an error.

* Minor update

Updated the readme to be concise with the other version.
Updated the setup script, removed the deprecated ownerId field.

* Consistency fix

The prefix for the bot in two files were different compared to the one in the set up script. Fixed for consistency.

* The real quick fix

Actually fixed the damn thing this time.
@YorkAARGH YorkAARGH merged commit 572c93f into master Aug 8, 2021
@YorkAARGH YorkAARGH deleted the v13-update branch August 8, 2021 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement guidebot The original version of guidebot. non-issue Not an issue, asking a question or making a statement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants