224420 - Documentation for new charting system

Status: RESOLVED FIXED

(Bugzilla :: Documentation, defect, P1)

Description

[Gervase Markham [:gerv]]

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

Gerv

Comment 1

[Gervase Markham [:gerv]]

Restriction which needs documenting: stored values contain public bugs only (see
bug 211164).

Gerv

Comment 2

[Gervase Markham [:gerv]]

Attachment: Patch v.1

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 3

[Gervase Markham [: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

Comment 4

[Gervase Markham [:gerv]]

Attachment: Patch v.2

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

Comment 5

[Dave Miller [:justdave]]

hmm, this patch doesn't exactly resolve the original description of this bug
though...

Comment 6

[Gervase Markham [:gerv]]

No; I'll be doing that as well if time permits.

Gerv

Comment 7

[Gervase Markham [: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

Comment 8

[Dave Miller [:justdave]]

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">
  

Comment 9

[Gervase Markham [:gerv]]

Fixed. Sorry. :-)

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

Gerv

Comment 10

[Vlad Dascalu]

FIXED.

Comment 11

[Vlad Dascalu]

Oops, sorry... Hurried about this one. :)

Comment 12

[Dave Miller [:justdave]]

removing the approval flag to avoid confusing vlad :)

Comment 13

[Gervase Markham [:gerv]]

Attachment: Proper Patch v.1

This is some actual documentation (at last :-) for both reporting and charting.


Gerv

Comment 14

[Gervase Markham [:gerv]]

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