Opened 9 years ago

Closed 8 years ago

#1368 closed defect (bug) (fixed)

Enhance notifications for ticket summary changes

Reported by: ocean90's profile ocean90 Owned by: ocean90's profile ocean90
Milestone: Priority: normal
Component: Communication (Matrix, Slack, IRC) Keywords:



In Slack:

The mail:

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: "WordPress Trac" <>
X-Trac-Version: 1.0.1
Precedence: bulk
Auto-Submitted: auto-generated
X-Mailer: Trac 1.0.1, by Edgewall Software
X-Trac-Project: WordPress Trac
Date: Sat, 31 Oct 2015 11:38:03 -0000
Subject: Re: [WordPress Trac] #34521: Unifying permission error messages
 (was: Translation strings - "You are not allowed to ...")
Message-ID: <>
References: <>
X-Trac-Ticket-ID: 34521
In-Reply-To: <>

#34521: Unifying permission error messages
 Reporter:  ramiy                  |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  Text Changes           |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |     Focuses:

Ticket URL: <>
WordPress Trac <>
WordPress publishing platform

Sadly the body doesn't contain a "Changes (by X)" line so we only have the "Subject:" line from the header.

Attachments (2)

1368-summary-only.png (67.4 KB) - added by ocean90 8 years ago.
1368-summary-and-other-changes.png (155.0 KB) - added by ocean90 8 years ago.

Download all attachments as: .zip

Change History (5)

This ticket was mentioned in Slack in #meta by ocean90. View the logs.

8 years ago

#2 @ocean90
8 years ago

In 3086:

Slack: Avoid two PHP notices when the Trac email doesn't contain a (Comment|Changes|Description changed) line.

See #1368.

#3 @ocean90
8 years ago

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

In 3087:

Slack: Improve notifications for ticket summary changes.

  • Parse the subject of a Trac mail because it's the only place where summary changes are mentioned.
  • Include ticket summary changes in the "changes" section.
  • Fall back to "Someone" if the author property isn't set. That's the case for summary only changes.

Fixes #1368.

Note: See TracTickets for help on using tickets.