Changeset 9228 for sites/trunk/api.wordpress.org/public_html/dotorg/slack/trac-bot.php

Timestamp:

10/24/2019 04:06:42 AM (5 years ago)

Author:

dd32

Message:

Slack: Thread trac-bot ticket responses.

When a ticket is referenced within a thread, trac-bot will no longer send it to the global channel (often wtihout any context).

There's an option to reply in-thread, but also to broadcast to the wider channel, but thought that best to avoid for now.
Unfortunately the slack outgoing-webhook implementation doesn't announce whether the threaded message we're being triggered by was broadcast or not.

Fixes #2555.

File:

• sites/trunk/api.wordpress.org/public_html/dotorg/slack/trac-bot.php (modified) (1 diff)

sites/trunk/api.wordpress.org/public_html/dotorg/slack/trac-bot.php

@@ -92 +92 @@
         }
 
-        $slack->send( $parser->get_channel() );
+        $slack->send( $parser->get_channel(), $parser->get_thread() );
 
         if ( $_POST['channel_name'] === 'test' ) {