WordPress.org

Making WordPress.org

Opened 5 weeks ago

Closed 4 weeks ago

#5446 closed defect (invalid)

Can keywords be added to the Trac>Slack integration for #ticket unfurling

Reported by: garrett-eclipse Owned by:
Milestone: Priority: normal
Component: Slack & IRC Keywords: has-screenshots
Cc:

Description

A thought to help bugscrubbers is to include the keywords on the unfurl when a ticket number is mentioned. In many cases the scrubbers main focus is to apply the correct keywords so displaying them without a click may help with this.

Attachments (2)

Screen Shot 2020-09-22 at 10.28.44 PM.png (144.7 KB) - added by garrett-eclipse 5 weeks ago.
The current unfurl when a ticket is mentioned on Slack.
Screen Shot 2020-09-24 at 10.31.06 am.png (83.7 KB) - added by dd32 4 weeks ago.

Download all attachments as: .zip

Change History (6)

@garrett-eclipse
5 weeks ago

The current unfurl when a ticket is mentioned on Slack.

#1 @dd32
5 weeks ago

Is the intention here to prevent the need to scrub tickets, or to provide context?

The reason I ask, is that displaying the keywords in Slack just so that it can be skipped to the next sounds like something that would be better off dealt with by filtering the report prior to scrubbing it - This would only add extra slackbot mentions to the ticket without actual discussion?

If it's wanted to provide context for discussion, would it make sense to remove one of the other pieces of information for space considerations?

Or would it make sense to combine the Type & Status fields into something like "open defect", "reopened enhancement" so that there's an even number of displayed fields? or are the keywords usually long enough that it'd probably take up the bottom two sections anyway?

#2 @garrett-eclipse
4 weeks ago

Thanks @dd32 it was mostly to provide context during the scrub to easier spot missing keywords and understand the status of the ticket.
In my experience some tickets can have alot of keywords so I would expect they could spread two columns as some in rare cases can have upwards of 10 (needs-patch, needs-testing, needs-privacy-review, needs-copy-review, has-screenshots, needs-dev-note, needs-design, needs-unit-tests, ...). In those longer cases having it in one column would make the message very long.

#3 @dd32
4 weeks ago

Turns out, keywords are included if the ticket has any keywords.

Focuses are also appended to the Component if the ticket has any focuses.

The lack of a keywords field is due to the ticket not having any keywords.

I'm inclined to leave it as is, rather than adding complexity by listing no-keywords.

#4 @garrett-eclipse
4 weeks ago

  • Resolution set to invalid
  • Status changed from new to closed

Well I'll be darned, sorry for the bother @dd32.

Note: See TracTickets for help on using tickets.