Maybe I'm the only person on earth who still prefers plain text emails most of the time. ;) But, it'd be relatively easy to add a config knob to let site admins decide if there should be an option on the forward form to select the email format you want to send someone. It'd all default to HTML, but if you go out of your way to allow it, you can give your site users the freedom to send what they prefer.

Comments

dww created an issue. See original summary.

dww’s picture

Status: Active » Needs review
StatusFileSize
new5.12 KB
dww’s picture

Issue summary: View changes
dww’s picture

Issue summary: View changes
StatusFileSize
new6.19 KB
new498 bytes

This adds forward_update_8200() to set the default for the new config knob (since this is a stable module and all that). ;)

p.s. Interdiff seems to be confused. Here's a raw diff of the two patch files.

dww’s picture

Issue summary: View changes

(Sorry, cut+paste error).

  • john.oltman committed 76fb5bc on 8.x-2.x authored by dww
    Issue #2969352 by dww: Add option to allow plain text emails.
    
john.oltman’s picture

Status: Needs review » Fixed
dww’s picture

Cool, thanks! :) I thought I might have to put up a little fight around this one. ;)

Cheers,
-Derek

john.oltman’s picture

No problem sir! I'm happy to add features for people willing to spend time testing and patching, especially ones that are relatively benign to existing installs.

dww’s picture

Status: Fixed » Needs review
StatusFileSize
new412 bytes

Whoops! This is what happens when I'm developing locally and using devel to handle mail. ;) Now that I deployed this to a real server to test it, I discovered this patch was missing a use statement it needs. Sorry.

  • john.oltman committed 995c5dc on 8.x-2.x
    Revert "Issue #2969352 by dww: Add option to allow plain text emails."...
john.oltman’s picture

Version: 8.x-2.x-dev » 8.x-3.x-dev
Status: Needs review » Fixed
dww’s picture

Sweet, thanks! Interesting that d.o doesn't show the commits to 8.x-3.x in this issue. Wonder what's up with that. Anyway, all looks good. Neither commit is in 8.x-2.x now, and both are in 8.x-3.x.

Yay,
-Derek

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.