Adding a discord adapter #223
Conversation
| @@ -104,10 +104,12 @@ | |||
| ; chat protocols | |||
| [irclj "0.5.0-alpha4"] | |||
| ;; use this fork which uses javax.websockets for compatability with Yetibot | |||
| [stylefruits/gniazdo-jsr356 "1.0.0"] | |||
| ;; gniazdo 1.2.2 needed for discljord | |||
| [stylefruits/gniazdo "1.2.2"] | |||
There was a problem hiding this comment.
Needed to update this for discord to work properly. I'm thinking this will end up breaking other things since this uses jetty's Websocket
There was a problem hiding this comment.
Seems like your local yb is running just fine? I'll do some more testing but this upgrade might be good to go.
There was a problem hiding this comment.
Yeah locally it is fine. I didn't test any other connectors though.
There was a problem hiding this comment.
I had to include this dep in the the yetibot/yetibotproject.clj as well to get it to run properly.
There was a problem hiding this comment.
Confirmed slack still works with this upgrade.
There was a problem hiding this comment.
Sick. Thanks for checking that out.
| @@ -104,10 +104,12 @@ | |||
| ; chat protocols | |||
| [irclj "0.5.0-alpha4"] | |||
| ;; use this fork which uses javax.websockets for compatability with Yetibot | |||
| [stylefruits/gniazdo-jsr356 "1.0.0"] | |||
| ;; gniazdo 1.2.2 needed for discljord | |||
| [stylefruits/gniazdo "1.2.2"] | |||
There was a problem hiding this comment.
Seems like your local yb is running just fine? I'll do some more testing but this upgrade might be good to go.
|
Note: adapters now pass the raw event in the chat-source so cmd handlers can optionally use platform-specific stuff, so we should do that for discord as well, e.g. https://github.com/yetibot/core/pull/224/files#diff-1a12310cc4d4bb44655929798120810eab79f8ad290cb8e65a3605d8fc3a34f0 |
Co-authored-by: Trevor Hartman <[email protected]>
Task:
Getting a discord adapter implemented.
I'm using the library from discljord to implement a lot of this.
Todo before merging: