Closed Bug 277615 Opened 20 years ago Closed 20 years ago

Update the "New Features" page for the 2.18 release

Categories

(Bugzilla :: bugzilla.org, defect)

Product:
Bugzilla
Component:
bugzilla.org
Version:
2.18
Type:
defect
Priority:
Not set
Severity:
blocker

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: mkanat, Assigned: mkanat)

References

(
URL
)

Details

Attachments

(1 file, 3 obsolete files)

Version 4
62.56 KB, patch
Details | Diff | Splinter Review 
The data for this page is in the Release Notes on bug 150149, it just needs to
be pulled out of there and HTML-ized.
I've updated the "New Features" list, and I've also placed the current bug
150149 copy of the Release Notes into the web version of their page.
Assignee: justdave → mkanat
Status: NEW → ASSIGNED
Attachment #170994 - Flags: review?
Comment on attachment 170994 [details] [diff] [review]
Updates for Feature List and Web Release Notes

> <h1>Features</h1>
> <ul>
> 	<li><a href="#gr">Generic Reporting</a></li>
>+        <li><a href="#gc">Generic Charting</a></li>
> 	<li><a href="#rs">Request System</a></li>
> 	<li><a href="#egs">Enterprise Group Support</a></li>
> 	<li><a href="#uwm">User Wildcard Matching</a></li>

>+<a name="glbs"></a>
>+<h2>Google-like Bug Search</h2>
>+
>+<p>Bugzilla now includes a very simple, Google-like &quot;Find a Specific Bug&quot; page,
>+in addition to its advanced search page.</p>
>+

"glbs" doesn't seem to have link at the top of the page. Other than that, it
looks good at my initial glance (assuming that the relnotes are identical to
the relnote bug :).
Attachment #170994 - Flags: review? → review-
Attached patch Version 2 (obsolete) — — Splinter Review 
Oh, thanks for catching that! I fixed this version. I also realized that
"Generic Charting" was in there twice, so I fixed that, and one typo, also.
Attachment #170994 - Attachment is obsolete: true
Attachment #171173 - Flags: review?
Comment on attachment 171173 [details] [diff] [review]
Version 2

>Index: releases/2.18/release-notes.html.tmpl
>@@ -241,7 +302,7 @@
>-- There are more &lt;link&gt; tags on the links toolbar for navigating quickly between
>+- There are more <link> tags on the links toolbar for navigating quickly between

The &lt; and &gt; etc need to stay.  That'll show up invisible in an HTML page
otherwise.

>@@ -250,7 +311,7 @@
>-  &lt;body&gt; tag on Bugzilla pages to allow custom end-user CSS to explicitly affect
>+  <body> tag on Bugzilla pages to allow custom end-user CSS to explicitly affect

>@@ -284,46 +480,1169 @@
>-- $::userid is gone in favor of Bugzilla-&gt;user-&gt;id
>+- $::userid is gone in favor of Bugzilla->user->id

>-- quietly_check_login() and confirm_login() are gone, use Bugzilla-&gt;login()
>+- quietly_check_login() and confirm_login() are gone, use Bugzilla->login()

>-- Use Bugzilla-&gt;user-&gt;login in place of $::COOKIE{Bugzilla_login}
>+- Use Bugzilla->user->login in place of $::COOKIE{Bugzilla_login}

>-- You can tell if there's a user logged in or not by checking if
>-  Bugzilla-&gt;user exists rather than looking for $::userid==0
>+- You can tell if there's a user logged in or not by using
>+  Bugzilla->user rather than looking for $::userid==0.
>+  In new 2.18 code, use defined(Bugzilla->user) && (Bugzilla->user->id)
>+  In 2.20, this will become just (Bugzilla->user->id)
>+  In templates, always test [% IF user.id %] rather than [% IF user %]
>+
>+- SendSQL() and related calls are deprecated, and the various $dbh methods 
>+  should be used instead, such as $dbh->prepare() and $dbh->execute(). 
>+  Bugzilla->dbh is the $dbh handle to use.

etc, there's probably more than that in the file
Attachment #171173 - Flags: review? → review-
Attached patch Version 3 (obsolete) — — Splinter Review 
OK, fixed the HTML entities with a big sed.
Attachment #171173 - Attachment is obsolete: true
Attachment #171185 - Flags: review?
Flags: blocking2.18?
Summary: Update the "New Features" page for the release → Update the "New Features" page for the 2.18 release
Flags: blocking2.18? → blocking2.18+
Comment on attachment 171185 [details] [diff] [review]
Version 3

the sentence about "see the Known Bugs section" needs to go away (that's in the
release notes, not on this page), but I'll fix that on checkin.
Attachment #171185 - Flags: review? → review+
one more patch coming, but I'll see it when bzbot announces it.  getting this
off my list in the meantime, as most of it's in. :)
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Attached patch Version 4 — — Splinter Review 
OK, this version removes that Known Bugs thing, and it also updates the Release
Notes to the ones that actually will ship with 2.18. :-)
Attachment #171185 - Attachment is obsolete: true
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: