Improve wizard spam
Bug #668320 reported by
Cédric Krier
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Odoo Server (MOVED TO GITHUB) |
Fix Released
|
Low
|
OpenERP's Framework R&D |
Bug Description
Here is a patch that improve the wizard:
- it adds a plain text version of the HTML
- it adds a message-id
- it detects if the message is in HTML or not
Related branches
lp:~openerp-dev/openobject-server/ssi-dev-framework
- OpenERP Core Team: Pending requested
-
Diff: 93 lines (+22/-4) (has conflicts)4 files modifiedbin/addons/base/res/partner/partner_view.xml (+1/-1)
bin/addons/base/res/partner/wizard/partner_wizard_spam.py (+5/-1)
bin/addons/base/res/res_log_view.xml (+1/-1)
bin/tools/misc.py (+15/-1)
Superseded
for merging
into
lp:~openerp-dev/openobject-server/trunk-dev-framework
- OpenERP Core Team: Pending requested
-
Diff: 84 lines (+16/-4)4 files modifiedbin/addons/base/res/partner/partner_view.xml (+1/-1)
bin/addons/base/res/partner/wizard/partner_wizard_spam.py (+5/-1)
bin/addons/base/res/res_log_view.xml (+1/-1)
bin/tools/misc.py (+9/-1)
Changed in openobject-addons: | |
importance: | Undecided → Wishlist |
Changed in openobject-addons: | |
assignee: | nobody → OpenERP R&D Addons Team 1 (openerp-dev-addons1) |
importance: | Wishlist → Low |
status: | New → Confirmed |
Changed in openobject-addons: | |
status: | Confirmed → In Progress |
Changed in openobject-addons: | |
milestone: | none → 6.0-rc2 |
affects: | openobject-addons → openobject-server |
Changed in openobject-server: | |
assignee: | OpenERP R&D Addons Team 1 (openerp-dev-addons1) → OpenERP's Framework R&D (openerp-dev-framework) |
milestone: | 6.0-rc2 → none |
status: | In Progress → Confirmed |
To post a comment you must log in.
Your suggested patch has been merged in server revision 3147 <email address hidden>.
I had to change a few things during the merge to adapt to latest code, but also improved the logic a bit to use a nested multipart/ alternative part when including both html and plaintext, instead of keeping them in the main multipart/mixed, which would cause most MUAs to display both parts one after the other.
Thanks for the patch!