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.

Sign up for GitHub

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

Jump to bottom

Fix: Making sure a command is there before checking permissions #88

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Fix: Making sure a command is there before checking permissions #88

merged 1 commit into from
Oct 30, 2024

Conversation

pdellaert
Copy link
Collaborator

Description

Making sure it only checks bot permissions once it knows for sure it will have to handle a command.

Discord Username

straks

@pdellaert pdellaert requested a review from benw202 October 30, 2024 20:31
@pdellaert pdellaert self-assigned this Oct 30, 2024
@pdellaert pdellaert added the skip changelog Skips the verify changelog check label Oct 30, 2024
@pdellaert pdellaert merged commit 8d8a805 into flybywiresim:staging Oct 30, 2024
3 of 4 checks passed
@pdellaert pdellaert deleted the dynamic-command-handler-permission-check branch October 30, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benw202 benw202 benw202 approved these changes

Assignees

@pdellaert pdellaert

Labels
skip changelog Skips the verify changelog check
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pdellaert @benw202