Closed Bug 58020 Opened 24 years ago Closed 21 years ago

Let whineatnews include descriptions with email

Categories

(Bugzilla :: Email Notifications, enhancement, P2)

Other
Other
enhancement

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: justdave, Assigned: pbaker)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
*** Bug 55597 has been marked as a duplicate of this bug. ***
Keywords: patch
Whiteboard: 2.14
Whiteboard: 2.14 → 2.16
Target Milestone: --- → Bugzilla 2.16
moving to real milestones...
Priority: P3 → P2
-> Bugzilla product, Email component, reassigning.
Assignee: tara → jake
Component: Bugzilla → Email
Keywords: review
Product: Webtools → Bugzilla
Whiteboard: 2.16
Version: other → unspecified
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
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
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.
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

Some other feature I'd like to see is the inclusion of the bug_severity since
this is not always clear from the summary.
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).
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 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+
*** Bug 122458 has been marked as a duplicate of this bug. ***
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+
Summary: Param to let whineatnews include descriptions with email → Let whineatnews include descriptions with email
wow i forgot all about this even. hehehe it's about time you guys got your act
together! ;-)
Attachment #18009 - Attachment is obsolete: true
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
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

Created:
Updated:
Size: