Edgewall Software

Opened 12 years ago

Closed 12 years ago

Last modified 9 years ago

#10885 closed enhancement (fixed)

Show ticket component in Timeline — at Version 5

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

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

API Changes:
Internal 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

Change History (7)

comment:1 by thomas.akesson@…, 12 years ago

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!

by thomas.akesson@…, 12 years ago

Screenshot of tickets with components and one without.

comment:2 by thomas.akesson@…, 12 years ago

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

by anonymous, 12 years ago

Attachment: timeline_comp_trunk.patch added

First attempt at a patch, ever.

comment:3 by thomas.akesson@…, 12 years ago

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 by Christian Boos, 12 years ago

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 by Christian Boos, 12 years ago

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

Applied with a few minor modification in r11411. Thanks!

Note: See TracTickets for help on using tickets.