WordPress.org

Making WordPress.org

Opened 8 months ago

Closed 8 months ago

#4053 closed defect (fixed)

Remove `/committers` command in Slack

Reported by: desrosj Owned by: dd32
Milestone: Priority: normal
Component: Slack & IRC Keywords: has-patch
Cc:

Description

The /committers command in slack has long been superseded by the @ committers group. It was recently accidentally used and a few problems came to light.

  1. It results in double pings because @ committers and the individual Slack handles are included.
  2. The list is out of date.

Attachments (2)

4053.diff (2.4 KB) - added by desrosj 8 months ago.
4053.2.diff (3.2 KB) - added by desrosj 8 months ago.

Download all attachments as: .zip

Change History (7)

@desrosj
8 months ago

#1 @desrosj
8 months ago

4053.diff removes /committers and updates the get_committers() list to be what I believe is currently accurate.

@desrosj
8 months ago

#2 @desrosj
8 months ago

4053.2.diff also updates core.trac so that the new committers have the Core Committer label on their comments.

#3 @dd32
8 months ago

In 8068:

Slack: Remove the /committers slash command handler as it's no longer needed.

The Slack slash integration has also been disabled.

Props desrosj.
See #4053.

#4 @dd32
8 months ago

In 8069:

Slack: Update the list of Committers for slash command whitelisting purposes.

Props desrosj, dd32.
See #4053.

#5 @dd32
8 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 8070:

Trac: Update the list of Committers to add their proper titles on Trac.

Props desrosj, dd32.
Fixes #4053.

Note: See TracTickets for help on using tickets.