Closed
Bug 297974
Opened 20 years ago
Closed 20 years ago
Add bug numbers to the generated RSS buglist
Categories
(Bugzilla :: User Interface, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: LpSolit, Assigned: LpSolit)
Details
Attachments
(1 file, 3 obsolete files)
|
1018 bytes,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
Since 2.19.3, when you run a query, buglist.cgi offers you to save your query as
a RSS entry. But this RSS menu only displays the summary of bugs. Displaying bug
numbers too would be fine.
Actual typical entry:
"Remove PostgreSQL support" (fake bug summary :-p )
Expected display:
"299999 - Remove PostgreSQL support" (for public bugs)
"[299999] - Remove PostgreSQL support" (for bugs being in at least one group)
Note the square brackets [..] if the bug is not public.
| Assignee | ||
Comment 1•20 years ago
|
||
Attachment #186540 -
Flags: review?(myk)
| Assignee | ||
Comment 2•20 years ago
|
||
Now of the form:
[Bug 299789] foo bar
and, for non public bugs:
[SEC] [Bug 299789] foo bar
Attachment #186545 -
Flags: review?(myk)
| Assignee | ||
Comment 3•20 years ago
|
||
Now:
Bug 578954 - Grant review to v3
If in a group:
@ Bug 578954 - Grant review to v3
As said already, this is my prefered solution ;)
| Assignee | ||
Updated•20 years ago
|
Attachment #186547 -
Flags: review?(myk)
| Assignee | ||
Comment 4•20 years ago
|
||
In all my patches, maybe should we set the 'non-public bug' symbol when
bug.secure_mode = "manual" only?
As RSS buglist only exists since 2.19.3, I think it would be fine to have this
for 2.20rc1.
Status: NEW → ASSIGNED
OS: Linux → All
Comment 5•20 years ago
|
||
> Bug 578954 - Grant review to v3
I understand that this syntax may seem easier to read, but what we do here
should be consistent with what we do for bugmail, which at the moment is:
[Bug 578954] Grant review to v3
As for "@" to signify non-public bugs, it's problematic, but I can't think of
anything better at the moment, so let's go with it.
| Assignee | ||
Comment 6•20 years ago
|
||
Attachment #186540 -
Attachment is obsolete: true
| Assignee | ||
Updated•20 years ago
|
Attachment #186545 -
Attachment is obsolete: true
Attachment #186547 -
Attachment is obsolete: true
Attachment #186656 -
Flags: review?(myk)
| Assignee | ||
Updated•20 years ago
|
Attachment #186540 -
Flags: review?(myk)
| Assignee | ||
Updated•20 years ago
|
Attachment #186547 -
Flags: review?(myk)
| Assignee | ||
Updated•20 years ago
|
Attachment #186545 -
Flags: review?(myk)
Updated•20 years ago
|
Attachment #186656 -
Flags: review?(myk) → review+
| Assignee | ||
Updated•20 years ago
|
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
Comment 7•20 years ago
|
||
Minor polish fix. Approved for checkin during 2.20 freeze.
Flags: approval? → approval+
| Assignee | ||
Comment 8•20 years ago
|
||
Checking in template/en/default/list/list.rss.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list.rss.tmpl,v <--
list.rss.tmpl
new revision: 1.2; previous revision: 1.1
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•