Edgewall Software
Modify

Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#10885 closed enhancement (fixed)

Show ticket component in Timeline

Reported by: thomas.akesson@… Owned by: thomas.akesson@…
Priority: normal Milestone: 1.1.1
Component: timeline Version:
Severity: normal Keywords: timeline ticket component
Cc:
Release Notes:

Optionally display the component of tickets in their timeline entries ([timeline] ticket_show_component setting).

API Changes:

Description

When a Trac instance covers many components:

  • some users might be more interested in the component they work with
  • ticket summary might not mention the component and must be places in context in order to be descriptive

There is a Hack for filtering the Timeline based on component, but unless filtering on just a single component it does not fully address the requirements for some users. http://trac-hacks.org/wiki/TimelineComponentFilterPlugin

Suggesting that the component is displayed in the Timeline if configuration ticket_show_component = true.

Example formatting: Ticket #123 (Component - Summary) updated by username

Attachments (2)

Screen Shot 2012-10-05 at 18.03.40.png (22.2 KB ) - added by thomas.akesson@… 6 years ago.
Screenshot of tickets with components and one without.
timeline_comp_trunk.patch (4.1 KB ) - added by anonymous 6 years ago.
First attempt at a patch, ever.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by thomas.akesson@…

I have a semi-completed patch for this which modifies the ITimelineEventProvider parts of ticket/web_ui.py. It extends data to also contain the component and then allows the rendering to use it if configured to.

Suggestions or objections before I submit the patch?

Thanks!

Changed 6 years ago by thomas.akesson@…

Screenshot of tickets with components and one without.

comment:2 Changed 6 years ago by thomas.akesson@…

Attached a screenshot showing an example. The top ticket was a test after removing all components to ensure that the code handled that gracefully.

Changed 6 years ago by anonymous

Attachment: timeline_comp_trunk.patch added

First attempt at a patch, ever.

comment:3 Changed 6 years ago by thomas.akesson@…

I cleaned up the patch and submitted while I was at it.

Feedback welcome, it was my first attempt at the Trac codebase. I made the assumption that it might make it into Trac 1.1.

comment:4 Changed 6 years ago by Christian Boos

Milestone: 1.1.1
Owner: set to Christian Boos

Looks good to me.

The variable naming should keep the style we have elsewhere (componentprefixcomponent_prefix), but we can take care of this.

comment:5 Changed 6 years ago by Christian Boos

Release Notes: modified (diff)
Resolution: fixed
Status: newclosed

Applied with a few minor modification in r11411. Thanks!

comment:6 Changed 6 years ago by Christian Boos

Owner: changed from Christian Boos to thomas.akesson@…

comment:7 Changed 4 years ago by Ryan J Ollos

#8802 closed as a duplicate.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain thomas.akesson@….
The resolution will be deleted.
to The owner will be changed from thomas.akesson@… to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.