-
-
Notifications
You must be signed in to change notification settings - Fork 9
Don't process comands until extensions are loaded #190
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
Conversation
✅ Deploy Preview for bot-core ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
487f04f to
b8bb6cf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This currently throws TypeError: object NoneType can't be used in 'await' expression
The .set method is not async, so you shouldn't await it since calling it returns None instead of a coro.
Hah yea, my bad. Fixed in a fixup |
shtlrs
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good, thanks.
b327ac0 to
ab03dbf
Compare
…s until all extensions are loaded.
550e9be to
0213817
Compare
|
Thanks Chris ! |
This PR adds
BotBase.wait_until_bot_startedwhich can be used to hold a process until all extensions are loaded.This will also wait until all app commands have synced, if
sync_app_commandswas set when callingBotBase.load_extensions.I have also overwritten
discord.ext.commands.Bot.process_commandsto first callpydis_core.BotBase.wait_until_bot_startedto ensure no commands are processed until all extensions are loaded.