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 QUIT: "channels" shouldn't contain all channels of the bot #359

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

christianrank
Copy link

channels should only contain channels where the quitting user was joined, not all the channels the bot is connected to

channels should only contain channels where the quitting user was joined, not all the channels the bot is connected to
@Zoddo
Copy link
Contributor

Zoddo commented Mar 19, 2016

Nobody to merge this PR?

@vBm
Copy link
Contributor

vBm commented Mar 19, 2016

@Zoddo Code doesn't match style for starters (brace is on it's own line while rest in the file it isn't, code should be consistent) would be my guess why it hasn't been merged yet.

ping @jirwin to shed some light on the topic :)

@jirwin
Copy link
Collaborator

jirwin commented Mar 26, 2016

Thanks for this. I merged #448 and I believe it covers the same issue.

@jirwin jirwin closed this Mar 26, 2016
@jirwin jirwin reopened this Mar 28, 2016
@davericher
Copy link

+1

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.

None yet

5 participants