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

Fix channel ignored when channel list is empty and blacklisted #509

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

rainbowdashlabs
Copy link
Owner

Checklist

  • I made changes to commands

    • I fully localised the command
    • I fully checked the commands functionality
  • I added new localisation codes

    • I fully translated it for all languages
    • I sorted properties alphabetically
  • I made changes to the database

    • I added my changed to a patch file
    • I made sure my database was up to date
    • I checked that the migration works
  • I made changes to internal structure

Short Description
For some reason stuff failed when the blacklist was enabled, but the channel list was empty, because we always returned false when the channel list was empty even when using the blacklist. After some consideration we noticed that this can be simplified by a lot.

@rainbowdashlabs rainbowdashlabs enabled auto-merge (squash) April 11, 2023 23:07
@rainbowdashlabs rainbowdashlabs merged commit ae5050a into development Apr 11, 2023
@rainbowdashlabs rainbowdashlabs deleted the fix/channel-blacklist branch April 11, 2023 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants