urlwatch icon indicating copy to clipboard operation
urlwatch copied to clipboard

SlackReporter with preformatted rich text

Open vimagick opened this issue 8 months ago • 2 comments

Hi, I've updated my code ( #776 )

Three things:

  • [x] Have you checked whether or not Mattermost supports Slack's rich text feature?

     Mattermost still use the original `{"text": text}`
    
  • [x] How about users who want to opt out of this new behavior and have the old (non-rich-text) behavior? Should there be an option? What should be the default?

    Enabled only when `rich_text: true`
    
  • [x] Please update CHANGELOG.md with information about this change

    Yes
    

vimagick avatar Dec 05 '23 14:12 vimagick

@vimagick Did you have time to look into these?

  • [ ] Can you add 'rich_text': False, to lib/urlwatch/storage.py in the 'slack': section, so that it's easily discoverable?
  • [ ] Can you add documentation for this to the Slack section in docs/source/reporters.rst

thp avatar Mar 12 '24 04:03 thp

@vimagick Needs rebasing, and there's still the two items from above. Thanks!

thp avatar Apr 24 '24 19:04 thp

Closing after no activity. Feel free to reopen and update the PR.

thp avatar Jul 30 '24 04:07 thp