Closed
Bug 1122125
Opened 10 years ago
Closed 10 years ago
Add a link to release notes from admin.cgi
Categories
(Bugzilla :: Administration, task)
Tracking
()
RESOLVED
FIXED
Bugzilla 5.0
People
(Reporter: dkl, Assigned: dkl)
References
Details
Attachments
(1 file, 1 obsolete file)
682 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
As part of bug 390955, the release notes link was removed from the home page citing that normal users normally should not care about that information.
We should add the link back to an admin page so that admins can get to it easily. In 5.0, an admin would need to know the exact URL to find them right now.
Not a blocker for 5.0 but should be fixed in a minor revision IMO.
dkl
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8549755 -
Flags: review?(LpSolit)
![]() |
||
Comment 2•10 years ago
|
||
Comment on attachment 8549755 [details] [diff] [review]
1122125_1.patch
>+ <dt id="release_notes" class="[% class %]">
It doesn't make sense to mark the link as not clickable if I'm not an admin. Everybody allowed to view this page should be free to access release notes. So we don't need a class here.
>+ <dd class="[% class %]">Detailed information related to this release of B[%%]ugzilla.</dd>
Same here. Moreover, remember that hardcoded "Bugzilla" is allowed in templates. No need for the [%%] hack anymore.
Attachment #8549755 -
Flags: review?(LpSolit) → review-
![]() |
||
Updated•10 years ago
|
Summary: Release notes should be linked to from somewhere the admin can see but not necessarily normal users. → Add a link to release notes from admin.cgi
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8549755 -
Attachment is obsolete: true
Attachment #8549841 -
Flags: review?(LpSolit)
![]() |
||
Comment 4•10 years ago
|
||
Comment on attachment 8549841 [details] [diff] [review]
1122125_2.patch
r=LpSolit
Attachment #8549841 -
Flags: review?(LpSolit) → review+
![]() |
||
Comment 5•10 years ago
|
||
I will let dkl request approval for 5.0 if he wants it there.
Flags: approval?
Assignee | ||
Comment 6•10 years ago
|
||
Very low impact and to me a minor regression anyway, so why not.
Flags: approval5.0?
Updated•10 years ago
|
Flags: approval?
Flags: approval5.0?
Flags: approval5.0+
Flags: approval+
![]() |
||
Updated•10 years ago
|
Target Milestone: --- → Bugzilla 5.0
Assignee | ||
Comment 7•10 years ago
|
||
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
8635fca..ac3f8d9 master -> master
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
a7c55f6..e3e7b71 5.0 -> 5.0
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•