Bug Details loads data from first record by default

RESOLVED FIXED

Status

Tree Management
OrangeFactor
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: mcote, Assigned: mcote)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Repro:

- go to http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=633566

OR

- go to http://brasstacks.mozilla.com/orangefactor/?display=Bug

and enter a bug number.

In both cases, the startday will default to 2010-08-01.  It should be the default (4 weeks ending today atm).

We probably haven't noticed this earlier because all the internal links to the Bug Details view include the date range.
(Assignee)

Comment 1

7 years ago
Might be cool to also display the date of the first recorded occurrence (with a special string if at beginning of records) along with a link to view the whole history (i.e. first occurrence to now).
(Assignee)

Updated

7 years ago
Assignee: nobody → mcote
Status: NEW → ASSIGNED
(Assignee)

Comment 2

7 years ago
Created attachment 521643 [details] [diff] [review]
Bug Details: default to standard date range, add "entire history" link

This makes the default dates on the Bug Details view the same as the other views (currently, the last 4 weeks). It also adds a link to see the entire history of the bug, from the day before the first appearance to the day after the last appearance.  I think it looks better with the extra days--though I also noticed that we handle 0 counts differently before and after a bug's first occurrence--but maybe that's how it should be.

Also had to fix calculateMetric() to actually do something with dataStartDate and dataEndDate.
Attachment #521643 - Flags: review?(jgriffin)
Comment on attachment 521643 [details] [diff] [review]
Bug Details: default to standard date range, add "entire history" link

Nice - I like the 'entire history' feature.
Attachment #521643 - Flags: review?(jgriffin) → review+
(Assignee)

Comment 4

7 years ago
Pushed as http://hg.mozilla.org/automation/orangefactor/rev/8447aba8bf1a, with a few corrections on top (left in some console logs).
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
You need to log in before you can comment on or make changes to this bug.