-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Don't process comands until extensions are loaded #190
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 |
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_started
which 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_commands
was set when callingBotBase.load_extensions
.I have also overwritten
discord.ext.commands.Bot.process_commands
to first callpydis_core.BotBase.wait_until_bot_started
to ensure no commands are processed until all extensions are loaded.