search

Leonidas-from-XIV / slacko

A neat interface for Slack
https://leonidas-from-xiv.github.io/slacko
Other
81 stars 11 forks source link

Support attachments in chat_post_message #11

Closed jerith closed 7 years ago

jerith commented 7 years ago

Slack messages can have attachments, as documented at https://api.slack.com/docs/message-attachments.

Adding attachment support would probably involve creating a new attachment type and adding an optional parameter of type attachment list to Slacko.chat_post_message.

jerith commented 7 years ago

I have a somewhat hacky attempt at an implementation over here: https://github.com/jerith/slacko/compare/ocaml-4.03...post-attachment

It builds on top of #12, so it doesn't make much sense to open a pull request before that is resolved. :-)

Leonidas-from-XIV commented 7 years ago

Yep, let's get #12 merged first, otherwise GitHub won't even let me write any review comments on this.

jerith commented 7 years ago

Fixed in #13.