web page names not viewable in tp series graphs (new front end)

VERIFIED FIXED in 0.6

Status

VERIFIED FIXED
11 years ago
2 years ago

People

(Reporter: anodelman, Assigned: lorchard)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

11 years ago
The name of each page loaded during tp is stored in the database (this goes for other tp-style tests like tdhtml, tsvg, tgfx, etc), but it isn't being displayed anywhere.  We only get an interval/value when hovering over a section of the graph.  In the old front end this was displayable and I think that it should find its way onto the page somewhere.
Assignee: nobody → lorchard
Target Milestone: --- → 0.4
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
Alice, do you know where this data was stored in the database? If it's already there this could be a quick(ish) change.
(Reporter)

Comment 2

10 years ago
The web page names are in dataset_extra_data.  They are keyed on dataset_id.  Each web page name is associated with an interval/time value much like the information in dataset_values.
Target Milestone: 0.4 → 0.5
Target Milestone: 0.5 → 0.6
(Assignee)

Comment 3

10 years ago
Created attachment 340623 [details] [diff] [review]
turning on the raw flag in data set, which causes extra data to appear

Okay, after pawing through the code and tracing various things around, the attached patch seems embarrassingly simple.  So, I'm assuming there's something wrong with it.

But, it does seem to cause the page names to appear on rollover in discrete graphs.
Attachment #340623 - Flags: review?(rdoherty)
Comment on attachment 340623 [details] [diff] [review]
turning on the raw flag in data set, which causes extra data to appear

Awesome! We'll have to see how you can top this :)
Attachment #340623 - Flags: review?(rdoherty) → review+
(Assignee)

Comment 5

10 years ago
checked in as changeset 128:0c70f2022551
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Created attachment 340650 [details]
Screenshot of it post-fix
Verified FIXED.
Status: RESOLVED → VERIFIED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.