abustany / mattermost-trac-bot Goto Github PK

Currently, the bot needs to be invited in the configured Mattermost channels before it is started. It could instead:

• Get the list of all public channels at startup, and try to join the ones it's not in
• Print a warning for all channels it cannot join, which we suppose are private channels
• Listen to "invitation" events at runtime to handle a post-startup invitation in a private channel

Trac sessions eventually expire.

We should probably check for 403 response on working trac, and reissue an authentication when it happens.

The bot should provide an alternative "compact" format when listing tickets, where all ticket numbers are listed on a single line (comma separated), and the summary/state/etc. are rendered instead in the title of the link (that would then appear in the tooltip when hovering it).

This setting should have a default value and be overridable per-channel.

It should be possible to define aliases for Trac instances, so for which would also be recognized when querying issues. For example, a Trac instance "admin" with an alias "ad" could be queried with "admin#123" and "ad#123".

When querying a non-existing ticket, the bot current says Unexpected HTTP status: 404. A more human-readable message like "no such ticket" would be better.

It's currently impossible to define different ticket templates for different Trac instances. This becomes problematic if not all Trac instances have the same fields. One should be able to define a global default, and to optionally refine it per Trac instance.