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

Slack: send_card() does not support markdown #42

Open
sheluchin opened this issue Aug 24, 2016 · 7 comments
Open

Slack: send_card() does not support markdown #42

sheluchin opened this issue Aug 24, 2016 · 7 comments

Comments

@sheluchin
Copy link

Using send_card() with the Slack backend does not render the given string to markdown prior to passing it to the underlying API.

For example:

self.send_card(                         
    in_reply_to=message,                
    body="`some text`",
)                                       

Results in the sent card actually containing

`some text`

rather than image in the usual Slack inline code style.

This applies to all of the markdown syntax that Slack supports.

The same issue also exists when using the fields argument of send_card().

@the0rem
Copy link

the0rem commented Sep 23, 2016

Slack states that markdown is disabled by default for attachments (https://api.slack.com/docs/message-formatting). It would be great to have this implemented to enable markdown for attachments.

@ghost
Copy link

ghost commented Dec 15, 2016

I would also enjoy this feature 👍

@vdemonchy
Copy link

Supporting this, it'd be great to have proper markdown support 👍

@DasFranck
Copy link

I support this too 👍

roidayan referenced this issue in roidayan/errbot Aug 15, 2017
Add markdown support for text and pre-text in slack cards.
Fixes #839

Signed-off-by: Roi Dayan <roi.dayan@gmail.com>
roidayan referenced this issue in roidayan/errbot Aug 15, 2017
Add markdown support for text and pretext in slack cards.
Fixes #839

Signed-off-by: Roi Dayan <roi.dayan@gmail.com>
@andrewthetechie
Copy link

@sheluchin if you have some time, could you check out errbotio/errbot#1284 and see if it addresses your request?

@sheluchin
Copy link
Author

@andrewthetechie That addresses most of the problem. I did create this ticket specifically because I was trying to put some markdown in the card.body, but using a field instead of the body would be an acceptable workaround for my use case.

I understand the difficulty of applying your fix to the card.body. I personally don't think it's worth the effort, because using a field is a good enough workaround.

Unless anyone feels strongly about it, I'll leave this issue open since it's not completely resolved. Maybe it will help someone figure things out at some point in the future.

Thanks very much for the fix!

@sijis sijis transferred this issue from errbotio/errbot Jul 23, 2021
@sijis
Copy link
Collaborator

sijis commented Jul 23, 2021

Related ticket too.
#28

@nzlosh nzlosh mentioned this issue Oct 17, 2021
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

No branches or pull requests

6 participants