pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] mass_mailing_(sms): Update UX

Browse Source 
Minor UX tweaks for more consistent and correct wording, visualization,
and editing ability for mass_mailing and mass_mailing_sms. Specifically:

- Fix all "This email is blacklisted for mass mailing" => ".. mass
  mailings" typos
- Make phone blacklist not editable (to align w/ email blacklist)
- Add "Archived" ribbon to phone and email blacklist
- Update menu items and actions
    - "Phone Blacklist" => "Blacklisted Phone Numbers"
    - "Blacklist" => "BLacklisted Email Addresses"

Supports task: 2117635
This commit is contained in:
Tiffany Chang (tic)
2020-03-25 15:58:42 +00:00
parent 469b1a85d0
commit 3a112c7b78
8 changed files with 15 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/crm/views/crm_lead_views.xml
+3 -3
View File
@@ -134,11 +134,11 @@
<field name="phone_state" invisible="1"/>
<label for="email_from" class="oe_inline"/>
<div class="o_row o_row_readonly">
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': ['|', ('is_blacklisted', '=', False), ('partner_address_email', '!=', False)]}" groups="base.group_user"/>
<field name="email_from"
attrs="{'invisible': [('partner_address_email', '!=', False)]}" string="Email" widget="email"/>
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': ['|', ('partner_is_blacklisted', '=', False), ('partner_address_email', '=', False)]}" groups="base.group_user"/>
<field name="partner_address_email"
attrs="{'invisible': [('partner_address_email', '=', False)]}" widget="email" string="Email"/>
@@ -156,7 +156,7 @@
<field name="phone_state" invisible="1"/>
<label for="email_from" class="oe_inline"/>
<div class="o_row o_row_readonly">
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': [('is_blacklisted', '=', False)]}" groups="base.group_user"/>
<field name="email_from" widget="email"/>
</div>
addons/mail/views/mail_blacklist_views.xml
+2 -1
View File
@@ -17,6 +17,7 @@
<field name="arch" type="xml">
<form string="Add Email Blacklist" duplicate="false" edit="false">
<sheet>
<widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
<group>
<group>
<field name="email"/>
@@ -44,7 +45,7 @@
</record>
<record id="mail_blacklist_action" model="ir.actions.act_window">
<field name="name">Blacklist</field>
<field name="name">Blacklisted Email Addresses</field>
<field name="res_model">mail.blacklist</field>
<field name="view_id" ref="mail_blacklist_view_tree"/>
<field name="search_view_id" ref="mail_blacklist_view_search"/>
addons/mail/views/res_partner_views.xml
+1 -1
View File
@@ -11,7 +11,7 @@
<field name="is_blacklisted" invisible="1"/>
<label for="email" class="oe_inline"/>
<div class="o_row o_row_readonly">
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': [('is_blacklisted', '=', False)]}" groups="base.group_user"></i>
<field name="email" widget="email"/>
</div>
addons/mass_mailing/views/mailing_contact_views.xml
+2 -2
View File
@@ -13,7 +13,7 @@
<field name="is_blacklisted" invisible="1"/>
<label for="contact_id" class="oe_inline"/>
<div class="o_row o_row_readonly">
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': [('is_blacklisted', '=', False)]}" groups="base.group_user"></i>
<field name="contact_id"/>
</div>
@@ -164,7 +164,7 @@
<group>
<label for="email" class="oe_inline"/>
<div class="o_row o_row_readonly" name="email_details">
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailing"
<i class="fa fa-ban" style="color: red;" role="img" title="This email is blacklisted for mass mailings"
aria-label="Blacklisted" attrs="{'invisible': [('is_blacklisted', '=', False)]}" groups="base.group_user"></i>
<field name="email" widget="email"/>
<field name="is_blacklisted" invisible="1"/>
addons/mass_mailing/views/mailing_mailing_views_menus.xml
+2 -2
View File
@@ -12,7 +12,7 @@
groups="mass_mailing.group_mass_mailing_user"/>
<!-- Configuration / Blacklist -->
<menuitem id="mail_blacklist_mm_menu" name="Blacklist"
<menuitem id="mail_blacklist_mm_menu" name="Blacklisted Email Addresses"
action="mail.mail_blacklist_action"
parent="mass_mailing_configuration"/>
@@ -21,4 +21,4 @@
name="Mass Mailing"
sequence="4"
parent="base.menu_custom"/>
</odoo>
</odoo>
addons/mass_mailing_sms/views/mailing_sms_menus.xml
+1 -1
View File
@@ -48,7 +48,7 @@
groups="mass_mailing.group_mass_mailing_user"/>
<!-- SMS Marketing / Configuration / Blacklist -->
<menuitem id="phone_blacklist_menu"
name="Phone Blacklist"
name="Blacklisted Phone Numbers"
parent="mass_mailing_sms_menu_configuration"
sequence="1"
action="phone_validation.phone_blacklist_action"
addons/phone_validation/models/mail_thread_phone.py
+1 -1
View File
@@ -33,7 +33,7 @@ class PhoneMixin(models.AbstractModel):
phone_blacklisted = fields.Boolean(
string='Phone Blacklisted', compute="_compute_phone_blacklisted", compute_sudo=True, store=False,
search="_search_phone_blacklisted", groups="base.group_user",
help="If the email address is on the blacklist, the contact won't receive mass mailing anymore, from any list")
help="If the phone number is on the blacklist, the contact won't receive mass mailing sms anymore, from any list")
@api.depends(lambda self: self._phone_get_number_fields())
def _compute_phone_sanitized(self):
addons/phone_validation/views/phone_blacklist_views.xml
+3 -2
View File
@@ -15,8 +15,9 @@
<field name="name">phone.blacklist.view.form</field>
<field name="model">phone.blacklist</field>
<field name="arch" type="xml">
<form string="Phone Blacklist" duplicate="false">
<form string="Phone Blacklist" duplicate="false" edit="false">
<sheet>
<widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
<group>
<group>
<field name="number"/>
@@ -44,7 +45,7 @@
</record>
<record id="phone_blacklist_action" model="ir.actions.act_window">
<field name="name">Phone Blacklist</field>
<field name="name">Blacklisted Phone Numbers</field>
<field name="res_model">phone.blacklist</field>
<field name="view_id" ref="phone_blacklist_view_tree"/>
<field name="search_view_id" ref="phone_blacklist_view_search"/>