Only loadScripts once

[technicalpickles]

This should fix slackapi/hubot-slack#210 over in hubot-slack. I'm not sure if there's any other implications of this, like if other adapters have workarounds for this or not.

[michaelansel]

First pass says this should be fine. I don't think we ever intentionally reload scripts.

[zapbuild]

When you are going to merge it into master branch? I'm getting the same issue "Hubot repeating himself twice after reconnect". I'm using hubot-slack adapter.

[technicalpickles]

Soon. I'm testing this branch in production, and hoping to verify the double-response is fixed.

[technicalpickles]

I still have this out in production. There weren't any reconnect events to verify this fixes it yet though 😓

[ThomasRooney]

I can verify that this has fixed my issue with multiple responses after a reconnect. Can we get it merged?

[Floby]

\o/
Le 12 nov. 2015 21:35, "Ben Lavender" notifications@github.com a écrit :

Merged #1069 #1069.

—
Reply to this email directly or view it on GitHub
#1069 (comment).

[Floby]

I take it this has been published with 2.17.0 ?

[bhuga]

Yes, this is in 2.17.0.

[Floby]

ok, thanks a lot =)

On 19 November 2015 at 15:52, Ben Lavender notifications@github.com wrote:

Yes, this is in 2.17.0.

—
Reply to this email directly or view it on GitHub
#1069 (comment).