Closed Bug 494037 Opened 15 years ago Closed 15 years ago

Release Notes for Bugzilla 3.4rc1

Categories

(Bugzilla :: Documentation, defect)

defect
Not set
blocker

Tracking

()

RESOLVED FIXED
Bugzilla 3.4

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file, 5 obsolete files)

The biggest thing we need for the 3.4rc1 release is Release Notes. This actually wasn't too hard for 3.2 (the Template Toolkit stuff makes it much easier), and I don't expect it to be impossibly hard for 3.4 either, particularly since 3.4 is such a smaller release than 3.2.
Flags: blocking3.4+
Hey manu, I know you mentioned that you wanted to work on relnotes. Are you working on these, or are you just working on the other smaller releases (which would be OK)? I just want to know, because we're just about ready to release 3.4rc1 and I'm going to do these release notes if you haven't started on them yet.
(In reply to comment #1)
>
> Hey manu, I know you mentioned that you wanted to work on relnotes. Are you
> working on these, or are you just working on the other smaller releases (which
> would be OK)?

I'll only be able to start on Tuesday so it's probably best you go ahead.
Assignee: documentation → mkanat
Attached patch WIP (obsolete) — Splinter Review
Here is a work in progress, containing most of the major features and code change notes.
Attached patch WIP 2 (obsolete) — Splinter Review
Added more features from bugs marked "relnote".
Attachment #387171 - Attachment is obsolete: true
Attached patch WIP 3 (obsolete) — Splinter Review
Now contains some of the Outstanding Issues.
Attachment #387173 - Attachment is obsolete: true
Attached patch v1 (obsolete) — Splinter Review
Attachment #387176 - Attachment is obsolete: true
Attachment #387336 - Flags: review?
Attachment #387336 - Flags: review? → review-
Comment on attachment 387336 [details] [diff] [review]
v1

>Index: template/en/default/pages/release-notes.html.tmpl

>+<h2><a name="v34_req">Minimum Requirements</a></h2>

This makes the title to be seen as an anchor. You must move </a> before the title.


>+<h3><a name="v34_feat_enter">Simple Bug Filing</a></h3>

Same problem here and for other <h3> titles.

Also, here and at several other places is "Bug" hardcoded, instead of $terms.Bug. This makes 009bugwords.t to fail.


>+<h3><a name="34_feat_see">"See Also" Field</a></h3>

The anchor name must start with "v34", not "34" alone.


>+  <li>When logging out, users are now redirected to the main page of
>+    [% terms.Bugzilla %] instead of an empty page.</li>

Must be [%+ ... %], else no whitespace is added before "Bugzilla". Either that or put "of" right before it.


>+  <li>If you input an invalid regular expression anywhere in
>+    [% terms.Bugzilla %], it will now tell you explicitly instead of failing
>+    cryptically.</li>

Same here.


>+<tr> <td>CGI</td>          <td class="req_new">3.21</td> </tr>

CGI 3.21 for Perl 5.8.x, but CGI 3.33 for Perl 5.10.x.


I didn't check all features one by one. I trust you listed them correctly. Everything looks good.
+<p>Users can now select what time zone they are in and Bugzilla will
+  adjust displayed times to be correct for their time zone. However,
+  time zones the user inputs are unfortunately still in 

times the user inputs

+<p>Here's an example: Let's say that we create a field called "Colors",
+  and we make the Product field "control the values" for Colors. Then we
+  add Blue, Red, Black, and Yellow as legal values for the "Colors" field.
+  Now we can say that "Blue" and "Red" only appear as valid choices in 
+  Product A, "Yellow" only appears in Product B, but "Black" <em>always</em>
+  appears.</p>

Maybe this paragraph can be dropped, or replaced by "see the docs for details"?
(In reply to comment #7)
> >+<h2><a name="v34_req">Minimum Requirements</a></h2>
> 
> This makes the title to be seen as an anchor. You must move </a> before the
> title.

  This is a broader general regression in 3.4 that we need to fix, not a problem with the release notes. (We need to add a CSS rule for a[name].)

(In reply to comment #8)
> Maybe this paragraph can be dropped, or replaced by "see the docs for details"?

  I don't think there are any docs. The problem is that the paragraph above is pretty much incomprehensible, even to me, and I wrote it. It's the nature of the logic of the thing, not the writing, I think.
Attached patch v2 (obsolete) — Splinter Review
Address LpSolit and Wurblzap's comments, fix bugwords test failures, and fix HTML validation.
Attachment #387336 - Attachment is obsolete: true
Attachment #387403 - Flags: review?
Attached patch Actual v2Splinter Review
Accidentally uploaded the wrong one.
Attachment #387403 - Attachment is obsolete: true
Attachment #387408 - Flags: review?
Attachment #387403 - Flags: review?
Attachment #387408 - Flags: review? → review+
Comment on attachment 387408 [details] [diff] [review]
Actual v2

Looks good. r=LpSolit
Flags: approval3.4+
Flags: approval+
I've checked in the release notes as they are, but I will still take review comments from anybody who wants to point out errors or improvements, and check in the fixes afterward.

tip:

Checking in docs/lib/Pod/Simple/HTMLBatch/Bugzilla.pm;
/cvsroot/mozilla/webtools/bugzilla/docs/lib/Pod/Simple/HTMLBatch/Bugzilla.pm,v  <--  Bugzilla.pm
new revision: 1.6; previous revision: 1.5
done
Checking in template/en/default/pages/release-notes.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/release-notes.html.tmpl,v  <--  release-notes.html.tmpl
new revision: 1.36; previous revision: 1.35
done

3.4:

Checking in docs/lib/Pod/Simple/HTMLBatch/Bugzilla.pm;
/cvsroot/mozilla/webtools/bugzilla/docs/lib/Pod/Simple/HTMLBatch/Bugzilla.pm,v  <--  Bugzilla.pm
new revision: 1.5.2.1; previous revision: 1.5
done
Checking in template/en/default/pages/release-notes.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/release-notes.html.tmpl,v  <--  release-notes.html.tmpl
new revision: 1.33.2.3; previous revision: 1.33.2.2
done
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: