Closed Bug 605412 Opened 14 years ago Closed 14 years ago

Remove "configure bugmail" from bugmail (newchangedmail)

Categories

(Bugzilla :: Email Notifications, enhancement)

3.6.2
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.2

People

(Reporter: Martijn.Ras, Assigned: Martijn.Ras)

Details

Attachments

(1 file)

Hello,

I receive lots of complaints from my users about the following part of the emails they receive:

  -- 
  Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email

I hacked these out of newchangedmail.txt.tmpl.

I think it would be useful if the content and layout could be controlled the administration interface.

Mazzel,

Martijn.
Why would an admin interface be useful? This would affect all users at once, and would bring back the old Bugzilla 2.x behavior. You could as well edit the template directly, as you did already. IMO, a better alternative would be to remove the line

 Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email

entirely. This URL doesn't need to be in each bugmail, and people who want to edit their email prefs can follow the URL to the bug, and then from here go to the prefs panel. mkanat?

I don't think it makes sense to make other parts of the email template optional.
Assignee: administration → email-notifications
Component: Administration → Email Notifications
Indeed, just removing the line is better ... no need to make things more complex than necessary.
Comment on attachment 484273 [details] [diff] [review]
Removes the line "Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email" from the templates.

Putting your patch in our radar.
Attachment #484273 - Flags: review?(mkanat)
Attachment #484273 - Flags: review?(LpSolit)
Summary: Make parts of newchangedmail.txt.tmpl optional and controlled through the administration interface → Remove "configure bugmail" from bugmail (newchangedmail)
Comment on attachment 484273 [details] [diff] [review]
Removes the line "Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email" from the templates.

Nice to see this URL which we never use going away. Thanks for the patch! :) r=LpSolit
Attachment #484273 - Flags: review?(mkanat)
Attachment #484273 - Flags: review?(LpSolit)
Attachment #484273 - Flags: review+
Assignee: email-notifications → Martijn.Ras
Status: NEW → ASSIGNED
Flags: approval+
Target Milestone: --- → Bugzilla 4.2
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/email/bugmail.html.tmpl
modified template/en/default/email/bugmail.txt.tmpl
Committed revision 7552.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
We should also remove the ---- ---- Stuff from the "You are receiving this email because" line now, though, since it doesn't have to be separated from anything anymore.
(In reply to comment #6)
> We should also remove the ---- ---- Stuff

Yeah, I thought about that too. :) Done.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/email/bugmail.txt.tmpl
Committed revision 7554.
We get a good number of complaints already for people not knowing where to change their e-mail settings. I worry that removing this will increase that... I guess we'll find out.
(In reply to comment #8)
> We get a good number of complaints already for people not knowing where to
> change their e-mail settings.

See, those people are unable to see the link when it's right under their nose. And the link annoys everybody who uses Bugzilla daily, because you change your email settings maybe once every two years. It's a good thing it goes away. :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: