diff options
| author | Thibault Delavallée <tde@odoo.com> | 2018-01-19 16:03:06 +0100 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2018-01-29 14:19:53 +0100 |
| commit | 11d2d30e91758baaa9c6241a86f97ae3dfe52ad2 (patch) | |
| tree | 7efa536b4a7cff0363559981803faef0504d8c96 /addons/mail/data/mail_data.xml | |
| parent | 27b30daea54b02c45edbd1b70434b77a86be73d6 (diff) | |
[REM] mail: remove follow / unfollow buttons in notification emails
Notification emails contain a link allowing to follow or unfollow
documents directly from the notification email. However there are
several issues with this button
* it require people to login to follow or unfollow documents;
* it is not linked to the email;
* if the customer does not have a login it is not possible to perform
the action;
* there is no confirmation step;
This commit removes the follow / unfollow button in the notification
emails. Indeed as people have the "Access document" button there is
only one more click to perform in order to subscribe or unsubscribe.
Concerning customers adding a complete control of the follow mechanism
would be costly in term of code and access rights checks. We therefore
decided to remove this button.
It also allows to simplify a bit the notification process as there
are less parameters to take into account when grouping recipients. It
allows to lessen the number of required template rendering when sending
notification emails.
Diffstat (limited to 'addons/mail/data/mail_data.xml')
| -rw-r--r-- | addons/mail/data/mail_data.xml | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/addons/mail/data/mail_data.xml b/addons/mail/data/mail_data.xml index ad6f58665c2..c30ba66b37b 100644 --- a/addons/mail/data/mail_data.xml +++ b/addons/mail/data/mail_data.xml @@ -91,18 +91,6 @@ <a href="${ctx['button_access']['url']}" style="padding: 8px 12px; font-size: 12px; color: #FFFFFF; text-decoration: none !important; font-weight: 400; background-color: #875A7B; border: 0px solid #875A7B; border-radius:3px">${ctx['button_access']['title']}</a> % endif - % if ctx.get('has_button_follow'): - % if ctx.get('has_button_access'): - | - % endif - <a href="${ctx['button_follow']['url']}" style="color: #875A7B; text-decoration: none !important;">${ctx['button_follow']['title']}</a> - % elif ctx.get('has_button_unfollow'): - % if ctx.get('has_button_access'): - | - % endif - <a href="${ctx['button_unfollow']['url']}" style="color: #875A7B; text-decoration: none !important;">${ctx['button_unfollow']['title']}</a> - % endif - % if ctx.get('actions'): % for action in ctx['actions']: | |