Closed
Bug 58020
Opened 24 years ago
Closed 21 years ago
Let whineatnews include descriptions with email
Categories
(Bugzilla :: Email Notifications, enhancement, P2)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: justdave, Assigned: pbaker)
References
Details
Attachments
(1 file, 1 obsolete file)
1.33 KB,
patch
|
jouni
:
review+
|
Details | Diff | Splinter Review |
Attaching a patch which adds a Param called "whineincludesdesc" which when turned on, will add the summary line from each bug under each of the URLs in the whine email.
Reporter | ||
Comment 1•24 years ago
|
||
Updated•23 years ago
|
Whiteboard: 2.14 → 2.16
Reporter | ||
Updated•23 years ago
|
Target Milestone: --- → Bugzilla 2.16
Reporter | ||
Comment 3•23 years ago
|
||
moving to real milestones...
Updated•23 years ago
|
Priority: P3 → P2
Comment 4•23 years ago
|
||
-> Bugzilla product, Email component, reassigning.
Assignee: tara → jake
Component: Bugzilla → Email
Keywords: review
Product: Webtools → Bugzilla
Whiteboard: 2.16
Version: other → unspecified
Comment 5•23 years ago
|
||
Why would this have to be a parameter? We've got parameter-itis. If it's good, let's do it. If not, let's not. Gerv
Reporter | ||
Comment 6•23 years ago
|
||
We are currently trying to wrap up Bugzilla 2.16. We are now close enough to release time that anything that wasn't already ranked at P1 isn't going to make the cut. Thus this is being retargetted at 2.18. If you strongly disagree with this retargetting, please comment, however, be aware that we only have about 2 weeks left to review and test anything at this point, and we intend to devote this time to the remaining bugs that were designated as release blockers.
Target Milestone: Bugzilla 2.16 → Bugzilla 2.18
Assignee | ||
Comment 7•23 years ago
|
||
This is a patch that adds summaries to the whineatnews email. This patch does NOT add a parameter to enable this feature. It is simply added since it's just a plain good idea anwyay. The other difference between this patch and the first patch is that this one puts the summary above the link to the bug, not below. The link is indented 2 extra spaces along with the arrow.
Assignee | ||
Comment 8•23 years ago
|
||
Sample output: [...snip...] Appended below are the individual URLs to get to all of your NEW bugs that haven't been touched for a week or more. You will get this message once a day until you've dealt with these bugs! Param to let whineatnews include descriptions with email -> http://bugzilla.mozilla.org/show_bug.cgi?id=58020 Customised resolutions. -> http://bugzilla.mozilla.org/show_bug.cgi?id=94534 component column should be after product column in query results -> http://bugzilla.mozilla.org/show_bug.cgi?id=101875
Comment 9•22 years ago
|
||
Some other feature I'd like to see is the inclusion of the bug_severity since this is not always clear from the summary.
Comment 10•22 years ago
|
||
Another thing: If the summary contains not only ASCII characters the default setting for the whinemail: pref needs to contain something like: Content-type: text/plain; format=flowed; charset=ISO-8859-1 Content-transfer-encoding: 8BIT in the header (ISO-8859-1 is probably a good default since it is the most spread setting default character encoding of MySQL).
Reporter | ||
Comment 11•22 years ago
|
||
Changing default owner of Email Notifications component to JayPee, a.k.a. J. Paul Reed (preed@sigkill.com). Jake will be offline for a few months.
Assignee: jake → preed
Comment 12•22 years ago
|
||
Comment on attachment 59423 [details] [diff] [review] add summaries to whineatnews email (without param) I'd call this a hack, but it works. The entire whineatnews is a hack anyway, so we might just as well apply this and clean up when we templatize. r=jouni
Attachment #59423 -
Flags: review+
Reporter | ||
Comment 13•21 years ago
|
||
*** Bug 122458 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 14•21 years ago
|
||
wow, a reviewed patch from 6 months ago that hasn't been checked in... and it probably hasn't bitrotted because nobody ever touches this file :) who wants to check it in?
Assignee: preed → pbaker
Flags: approval+
Reporter | ||
Updated•21 years ago
|
Summary: Param to let whineatnews include descriptions with email → Let whineatnews include descriptions with email
Assignee | ||
Comment 15•21 years ago
|
||
wow i forgot all about this even. hehehe it's about time you guys got your act together! ;-)
Reporter | ||
Updated•21 years ago
|
Attachment #18009 -
Attachment is obsolete: true
Reporter | ||
Comment 16•21 years ago
|
||
Checking in whineatnews.pl; /cvsroot/mozilla/webtools/bugzilla/whineatnews.pl,v <-- whineatnews.pl new revision: 1.9; previous revision: 1.8 done
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•