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

Fixed issue #218 #219

Closed
wants to merge 3 commits into from
Closed

Conversation

sindastra
Copy link
Contributor

I fixed issue #218, a regular expression, a typo and updated the return messages.

Red-M and others added 3 commits October 27, 2015 09:39
Fixed regular expression
Fixed typo
Updated return messages to include the channel
@daboross
Copy link
Contributor

daboross commented Jun 16, 2016

I'm going to slightly change the wording here: the current message implies that the last time the person talked in any channel was in the specified channel, while that's not at all what the command is returning.

Besides that, thanks! PR will be merged.

@daboross
Copy link
Contributor

Merged in 601e9a6.

@daboross daboross closed this Jun 16, 2016
linuxdaemon pushed a commit to linuxdaemon/CloudBot that referenced this pull request Nov 14, 2017
…-fix

Properly start the botwide observer on start
linuxdaemon added a commit to linuxdaemon/CloudBot that referenced this pull request Mar 30, 2018
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.

4 participants