Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#8695 closed enhancement (fixed)

burndown graph should not finish with milestone's 'due' date but with 'completed' date

Reported by: falkb Owned by: osimons
Priority: normal Component: EstimationToolsPlugin
Severity: normal Keywords:
Cc: osimons Trac Release: 0.12

Description

I have a burndown graph

 [[BurndownChart(milestone=Foo, startdate=2011-03-10)]]

in a milestone that was not finished until the 'due' date of the milestone. Unfortunately, I cannot see the development of the remaining hours from that 'due' date until today, because the Burndown graph does end with the 'due' date.

That is why the burndown graph should display the range until the 'completed' date of the milestone, maybe in a red colour between 'due' and 'completed' date to mark it as overdue.

Attachments (1)

t8695-no_due_in_past-r10048.diff (1.1 KB) - added by osimons 3 years ago.
Patch to not use milestone 'due' if it is in the past.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 3 years ago by osimons

  • Owner changed from hoessler to osimons

Oh, I see - it will only display the time between 'due' and 'completed' AFTER the milestone is actually completed. So when checking 'due' we should not actually use it unless it is a timestamp in the future. If 'due' is in the past, we just use the current 'now' as enddate.

Makes sense. Should be a simple patch. I'll look into it.

Changed 3 years ago by osimons

Patch to not use milestone 'due' if it is in the past.

comment:2 in reply to: ↑ description ; follow-up: Changed 3 years ago by anonymous

Patch works! Thanks.

P.S.:
Replying to falkb:

... the burndown graph should display ... in a red colour between 'due' and 'completed' date to mark it as overdue.

Or maybe a vertical red line at 'due' date to show we've exceeded the limit

comment:3 in reply to: ↑ 2 ; follow-up: Changed 3 years ago by osimons

Replying to anonymous:

Or maybe a vertical red line at 'due' date to show we've exceeded the limit

This is really a separate request, as the plugin does not yet have any means of tagging custom events or otherwise drawing anything other than the line. #4131 and #4217 also requests some variation markings, so please transfer your ideas to one of those tickets or create a new ticket for it. This particular 'due' vs 'completed' vs 'today' as value for enddate is just one possible use for such a general marking-feature.

I'll get the patch for enddate calculation committed, though.

comment:4 Changed 3 years ago by osimons

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

(In [10082]) EstimationToolsPlugin: Don't use milestone 'due' date if it falls in the past, as data between 'due' and 'today' wont then be charted.

Fixes #8695. Thanks to falkb for report.

comment:5 in reply to: ↑ 3 Changed 3 years ago by falkb

Replying to osimons:

Replying to anonymous:

Or maybe a vertical red line at 'due' date to show we've exceeded the limit

#4131 and #4217 also requests some variation markings

#4131 would be a good solution for showing the overdue 'due' date also. So I wont create a new ticket.

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from osimons. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.