mattermost-plugin-github icon indicating copy to clipboard operation
mattermost-plugin-github copied to clipboard

Published 20 hours ago • verified publisher icon mattermost

Add per-subscription setting to collapse all posts

Open mickmister opened this issue 3 years ago • 3 comments

Depending on the kind of event that occurs, the plugin will expand the description sometimes (for new issues and new PRs), and collapse the description sometimes (for closing an issue or adding labels). The expanded description can cause a lot of noise in a channel, so we should add a setting to disable the expanded description.

The task here is to introduce an optional subscription parameter "collapsed", to have all posts be collapsed.

Issue created from a Mattermost message by @mickmister.

mickmister avatar Sep 27 '21 21:09 mickmister

We should also unconditionally put the expanded issue/PR in the post's "card" data for all events. This is done by inserting the markdown into post.props.card. More info on this here

mickmister avatar Sep 28 '21 14:09 mickmister

Hi @rinkimekari, would you like to be assigned to this ticket?

mickmister avatar Jan 26 '22 21:01 mickmister

Hi @rinkimekari, would you like to be assigned to this ticket?

I'd be happy to! There are just a few things with the problems with --exclude that I need to figure out. Shouldn't be too much longer till I have a PR ready.

rinkimekari avatar Jan 27 '22 01:01 rinkimekari

Resolved with https://github.com/mattermost/mattermost-plugin-github/pull/534

mickmister avatar Aug 29 '22 16:08 mickmister