Mattermost-plugin-github: Use custom post type to display TODO list

Created on 4 Jun 2019  路  1Comment  路  Source: mattermost/mattermost-plugin-github

Currently, the TODO list that gets posted daily or by /github todo looks something like this:

Screen Shot 2019-06-04 at 16 41 25

This works ok but it's hard to read, is only minimally functional and I think we can add something that does way better. Plugins have a cool feature that lets you use custom post types to render your own React components in place of the regular post components. Leveraging this we can:

  1. Display way more information about the pull requests/issues upfront in a much more readable way
  2. Add more functionality, such as a button to open all the unread messages so users don't have to click through each one

I started working on this in the unreads-v2 branch with the following commit https://github.com/mattermost/mattermost-plugin-github/commit/54e8ecf1f797ad517f362e40424cea4b78eba371 but stopped for two reasons: 1) I ran out of time and 2) plugins could not use custom post types for ephemeral messages (which the /github todo command returns). As of the next Mattermost version 5.12 coming out soon, 2) is possible thanks to https://github.com/mattermost/mattermost-webapp/pull/2759. Unfortunately, 1) is still the case as I'm still a bit too busy to work on it.

If someone is interested in picking up where I left off I think this would be a really great feature that would take the GitHub plugin to the next level. It'll require a bit of everything from server side, front end, UX design and merging master into my branch.

Here's what it was looking like when I stopped:

Screen Shot 2018-11-11 at 15 55 15

Easy Good First Issue Help Wanted TecGo TecReactJS TypEnhancement Up For Grabs

Most helpful comment

@jwilander just an FYI, part of what this PR does wass already implemented in another PR merged recently: https://github.com/mattermost/mattermost-plugin-github/pull/111 . Not as pretty as your version, but it solves the problem of not knowing what the PRs/Issues are :)

>All comments

@jwilander just an FYI, part of what this PR does wass already implemented in another PR merged recently: https://github.com/mattermost/mattermost-plugin-github/pull/111 . Not as pretty as your version, but it solves the problem of not knowing what the PRs/Issues are :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jfrerich picture jfrerich  路  4Comments

mattermod picture mattermod  路  6Comments

ThiefMaster picture ThiefMaster  路  6Comments

rien picture rien  路  4Comments

mattermod picture mattermod  路  6Comments