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

Upgrade botkit to 0.6.13 #141

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Upgrade botkit to 0.6.13 #141

wants to merge 3 commits into from

Conversation

samjbobb
Copy link
Contributor

We were having trouble with lookerbot disconnecting and not reconnecting every few days, as described in #118 .

I blindly updated botkit to 0.6.13.

We haven't needed to restart since approximately April 20 (~ 5 weeks). We haven't run into any errors.

Looks like the tests still pass. I haven't investigated throughly for breaking changes between botkit 0.0.15 and 0.6.13. It's possible this breaks something in lookerbot that we're just not using.

However, it seems like it might be a fix for the pretty painful issue #118 .

Hope this helps.

@hporter
Copy link

hporter commented Jul 31, 2018

We are testing this - seems to work well so far so thanks! Will continue to monitor.

Merging this will save a lot of time for a lot of people I imagine if #118 is solved.

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