Documentation for new charting system

RESOLVED FIXED in Bugzilla 2.18

Status

()

defect
P1
major
RESOLVED FIXED
16 years ago
7 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

2.17.4
Bugzilla 2.18

Details

Attachments

(1 attachment, 2 obsolete attachments)

Bug 16009 produced a new charting system; it needs documenting.

Gerv
Severity: normal → major
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.18
Restriction which needs documenting: stored values contain public bugs only (see
bug 211164).

Gerv
Posted patch Patch v.1 (obsolete) — Splinter Review
This adds a note to the series creation screen which says that only public bugs
will be recorded. I also plan to add a section to the documentation about
charting as time permits.

Gerv
Comment on attachment 139259 [details] [diff] [review]
Patch v.1

Dave - this is an HTML text change only. Review and approve for 2.17.7?

Gerv
Attachment #139259 - Flags: review?(justdave)
Posted patch Patch v.2 (obsolete) — Splinter Review
You wouldn't have thought I could mess up a patch like this, would you? <sigh>
Here's one with clearer wording. Review request still applies.

Gerv
Attachment #139259 - Attachment is obsolete: true
hmm, this patch doesn't exactly resolve the original description of this bug
though...
No; I'll be doing that as well if time permits.

Gerv
Checking in template/en/default/search/search-create-series.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/search/search-create-series.html.tmpl,v
 <--  search-create-series.html.tmpl
new revision: 1.5; previous revision: 1.4
done

Gerv
Gerv: The check-in from comment #7 causes the tests to fail on Tinderbox.

not ok 8 - template/en/default/search/search-create-series.html.tmpl contains
invalid bare words (e.g. 'bug') --WARNING
#     Failed test (t/009bugwords.t at line 90)
33: 

<p style="color: red">
  Note: there is currently a restriction that data sets will only count public
  bugs (those not in any group).
</p>

<form method="get" action="chart.cgi" name="chartform">
  
Fixed. Sorry. :-)

/me adds a reminder to run tests to his checkin script.

Gerv
FIXED.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Oops, sorry... Hurried about this one. :)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
removing the approval flag to avoid confusing vlad :)
Status: REOPENED → NEW
Flags: approval+
Component: Reporting/Charting → Documentation
This is some actual documentation (at last :-) for both reporting and charting.


Gerv
Attachment #139260 - Attachment is obsolete: true
Fixed.

Checking in docs/xml/using.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v  <--  using.xml
new revision: 1.19; previous revision: 1.18
done

Gerv
Status: NEW → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.