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

No reply to bots #1927

Merged

Conversation

AutumnClove
Copy link
Member

@AutumnClove AutumnClove commented Jul 17, 2019

After creating your pull request, tick these boxes if they are applicable to you.

  • I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
  • I have tested my changes on Python 3.5.3 or higher

Description

Adds a check to ignore other bots with an id override
Thanks @TheerapakG for showing typos and spoon feeding processing id's (TheerapakG: I did not tell you how to do id processing at all. I only say that it is needed. I did not spoonfeeding)

Related issues (if applicable)

@AutumnClove AutumnClove changed the title Merge pull request #40 from Just-Some-Bots/review No reply to bots Jul 17, 2019
@AutumnClove AutumnClove marked this pull request as ready for review July 17, 2019 03:45
@AutumnClove AutumnClove added a/options affects the bot's options system t/enhancement [PRs] Someone did a thing labels Jul 17, 2019
musicbot/config.py Show resolved Hide resolved
musicbot/bot.py Outdated Show resolved Hide resolved
@AutumnClove AutumnClove added the a/frontend Affects the frontend (visual appearance/ feel of the bot) label Jul 17, 2019
@jayktaylor jayktaylor merged commit af87d5e into Just-Some-Bots:review Jul 23, 2019
@AutumnClove AutumnClove deleted the improvement-no-reply-bots branch July 24, 2019 02:57
TheerapakG added a commit to TheerapakG/MusicBot that referenced this pull request Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a/frontend Affects the frontend (visual appearance/ feel of the bot) a/options affects the bot's options system t/enhancement [PRs] Someone did a thing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants