Changeset 1787

Timestamp:

07/26/2015 06:55:54 PM (9 years ago)

Author:

nacin

Message:

Migrate @committers to /committers.

This will allow committer names to be included in the message, thus triggering a push notification with the actual content.

Location:

sites/trunk/api.wordpress.org/public_html/dotorg/slack

Files:

• announce.php (modified) (1 diff)
• committers.php (modified) (1 diff)

sites/trunk/api.wordpress.org/public_html/dotorg/slack/announce.php

@@ -27 +27 @@
 }
 
-if ( $_POST['token'] !== WEBHOOK_TOKEN && $_POST['token'] !== WEBHOOK_TOKEN_ALT ) {
-    return;
+$i = 0;
+// WEBHOOK_TOKEN_1, WEBHOOK_TOKEN_2, etc.
+while ( defined( __NAMESPACE__ . '\\WEBHOOK_TOKEN_' . ++$i ) ) {
+    if ( constant( __NAMESPACE__ . '\\WEBHOOK_TOKEN_' . $i ) === $_POST['token'] ) {
+        run( $_POST );
+    }
 }
 
-run( $_POST );
+}
 
-}

sites/trunk/api.wordpress.org/public_html/dotorg/slack/committers.php

@@ -11 +11 @@
 }
 
-// These should be Slack usernames.
-switch ( $_POST['channel_name'] ) {
-    case 'bbpress' :
-    case 'buddypress' :
-    case 'glotpress' :
-        return;
-    default:
-        $committers = array(
-            'westi', 'azaozz', 'nb', 'josephscott', 'dd32', 'nacin', 'koop', 'duck_',
-            'helen', 'sergey', 'ocean90', 'wonderboymusic', 'drew', 'mark', 'johnbillion',
-            'kovshenin', 'jorbin', 'boone', 'jeremyfelt', 'pento', 'obenland', 'iseulde',
-            'westonruter',
-        );
-        break;
-}
+echo json_encode( array(
+    'username'   => 'wordpressdotorg',
+    'link_names' => 1,
+    'text'       => sprintf( '@%s: Use the `/committers` command.', $_POST['user_name'] ),
+) );
 
-if ( in_array( $_POST['user_name'], $committers, true ) ) {
-    $notify = array_diff( $committers, array( $_POST['user_name'] ) );
-    $notify = '(cc: @' . implode( ' @', $notify ) . ')';
-    echo json_encode( array( 'text' => $notify, 'username' => $_POST['user_name'], 'link_names' => 1 ) );
-}
+exit;