Closed Bug 417531 Opened 12 years ago Closed 12 years ago

Release Notes for 3.0.4

Categories

(Bugzilla :: Documentation, defect, blocker)

defect
Not set
blocker

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: emmanuel, Assigned: cedric.corazza)

References

()

Details

Attachments

(1 file, 6 obsolete files)

 
Assignee: general → cedric.corazza
Attached patch Patch for 3.0.4 Releases notes (obsolete) — Splinter Review
Attachment #311245 - Flags: review?(mkanat)
Attachment #311245 - Attachment is obsolete: true
Attachment #311363 - Flags: review?(LpSolit)
Attachment #311245 - Flags: review?(mkanat)
Comment on attachment 311363 [details] [diff] [review]
Patch for 3.0.4 Release notes - v2

Couple of minor points I noticed when looking through it:

>+<ul>
>+  <li>Bugzilla::Update never updated bugzilla-update.xml if the file already
>+    exists locally, thus Bugzilla administrators were never noticed about new
>+    releases.

noticed -> notified

>+  <li>A new CSS was added to gather all site customizations and ease the maintenance
>+    of a customized Bugzilla installation.
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=251740">[% terms.Bug %] 251740</a>)</li>

A new CSS *file*
Comment on attachment 311363 [details] [diff] [review]
Patch for 3.0.4 Release notes - v2

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

>+  <li>Bugzilla::Update never updated bugzilla-update.xml if the file already
>+    exists locally, thus Bugzilla administrators were never noticed about new
>+    releases.

s/noticed/notified/ as Colin said.


>+  <li>There were unnecessary line breaks and excessive indentation in email
>+    subject lines.
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=374424">[% terms.Bug %] 374424</a>)</li>

This is the wrong bug number. It's bug 411544.


>+  <li>'Test' in mail_delivery_method didn't generate a valid mbox file preventing
>+    using it for QA tests.

Not sure this one is important enough to be listed here.


>+  <li>A new CSS was added to gather all site customizations and ease the maintenance
>+    of a customized Bugzilla installation.

Also not sure this one should be listed here. Also, it's a new CSS *hook*.


Should be listed:

- Many bug fixes about email_in.pl (92274, 377025, 412943, 413672).
- You could previously clear the resolution of resolved bugs, without reopening them (424380).
- You could close a bug as VERIFIED and CLOSED with no resolution (418863).
- merge_users.pl now correctly rederives group memberships based on regexps when merging two user accounts (422111).
- checksetup.pl now correctly upgrades installations containing tables using Latin1 encoding, UTF-8 or a mix of both (374951).
- Comments written in CJK languages are now correctly displayed (388723).
- On Windows, temporary files created when uploading attachments are now correctly deleted when the upload is complete (414002).
- Many improvements in the documentation.


>+  For details, see the
>+  <a href="http://www.bugzilla.org/security/3.0.4/">Security Advisory</a></p>

The URL is wrong. Should be 3.0.3/ instead of 3.0.4/.
Attachment #311363 - Flags: review?(LpSolit) → review-
> s/noticed/notified/ as Colin said.

fixed

> This is the wrong bug number. It's bug 411544

fixed

> Not sure this one is important enough to be listed here.

It has the relnote keyword for trunk. I let it for now.

> Also not sure this one should be listed here. Also, it's a new CSS *hook*.

It has the relnote keyword for trunk. I added 'hook' and let it for now

> Should be listed:
> ...

Added

> The URL is wrong. Should be 3.0.3/ instead of 3.0.4/.

fixed
Attachment #311363 - Attachment is obsolete: true
Attachment #311831 - Flags: review?(LpSolit)
Comment on attachment 311831 [details] [diff] [review]
Patch for 3.0.4 Release notes - v3

>+  <li>There were many bug fixes about email_in.pl.
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=92274">[% terms.Bug %] 92274</a>),
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=377025">[% terms.Bug %] 377025</a>),
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=412943">[% terms.Bug %] 412943</a>),
>+    (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=413672">[% terms.Bug %] 413672</a>)

Please remove parens between each bug number. Only keep the very first one and very last one. Also, there is a missing </li> at the end.

r=LpSolit with this comment addressed. Please upload a new patch and carry forward my r+.
Attachment #311831 - Flags: review?(LpSolit) → review+
Fixed. forwarding r+
Attachment #311831 - Attachment is obsolete: true
Attachment #312286 - Flags: review+
Blocks: 428383
No longer blocks: 428383
Fixing security advisory directory
Attachment #312286 - Attachment is obsolete: true
Attachment #317347 - Flags: review?(LpSolit)
Comment on attachment 317347 [details] [diff] [review]
Patch for 3.0.3 Release notes - v5

>   à partir d'une version précédente</a>. Si vous effectuez une mise à jour à partir d'une version antérieure
>   à la version 2.22, assurez-vous de lire toutes les
>   <a href="#v30_previous">notes de versions précédentes</a> entre votre version

Please attach the *english* release notes. :)
Attachment #317347 - Flags: review?(LpSolit) → review-
Sorry for the previous mess..
Attachment #317347 - Attachment is obsolete: true
Attachment #319037 - Flags: review?(LpSolit)
Attached patch v6Splinter Review
Okay, here's the relnotes with some updates.
Attachment #319037 - Attachment is obsolete: true
Attachment #319308 - Flags: review?(LpSolit)
Attachment #319037 - Flags: review?(LpSolit)
Comment on attachment 319308 [details] [diff] [review]
v6

Maybe mentioning that process_bug.cgi is more robust when resolving/reopening bugs would make sense, but maybe is that not visible enough as you have to hack the URL in a non-obvious way. Anyway, r=LpSolit
Attachment #319308 - Flags: review?(LpSolit) → review+
tip:

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.16; previous revision: 1.15

3.0 branch:

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.1.2.11; previous revision: 1.1.2.10
done
Status: NEW → RESOLVED
Closed: 12 years ago
Component: Bugzilla-General → Documentation
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 3.0
You need to log in before you can comment on or make changes to this bug.