Opened 6 years ago
Closed 6 years ago
#3799 closed defect (bug) (fixed)
`/announce` and `/here` do not work for newer users
Reported by: | dd32 | Owned by: | dd32 |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Communication (Matrix, Slack, IRC) | Keywords: | |
Cc: |
Description
As reported in Slack #meta https://wordpress.slack.com/archives/C02QB8GMM/p1536701029000100 some users are unable to use the /announce
and /here
commands despite being whitelisted.
This is because Slack changed awhile ago to deprecate the user_name
field: https://api.slack.com/changelog/2017-09-the-one-about-usernames
As done elsewhere in our Slack API code, the proper thing to do here is to determine the user by the slack user_id
and convert that back into a WordPress user_login
before comparing it to the whitelists.
Change History (4)
Note: See
TracTickets for help on using
tickets.
In 7661: