search

klothoplatform / slack-notifier

Slack notification bot
MIT License
3 stars 0 forks source link

markdown in a title can the top-level post's links #44

Open ghost opened 1 year ago

ghost commented 1 year ago

For example, klothoplatform/sample-apps#31 has the following title:

add `npm run <klotho | pulumi>` scripts

This got rendered as:

PR <https://github.com/klothoplatform/sample-apps/pull/31%7C#31: add npm run <klotho | pulumi> scripts> (+777/-0) by yuval-klotho

%7C is |, weirdly enough.

Anyway, we should properly escape these things. Would be good to write a general markdown-to-slack conversion (including escaping any slack-specific sequences). Related to #12.