diff options
| author | Rémi Rahir <rar@odoo.com> | 2020-08-18 09:51:43 +0000 |
|---|---|---|
| committer | Rémi Rahir <rar@odoo.com> | 2020-08-18 13:53:46 +0000 |
| commit | 08ed6afa2a6e509b834e1c5fa9f732574c8b2162 (patch) | |
| tree | d89a1991a7232fb9274405342c8724e103c7cd9d /addons/partner_autocomplete/static | |
| parent | cef57b930b27cb487973f251797c93ea3967a5b3 (diff) | |
[IMP] partner_autocomplete: do not fill email field on partner
- the email returned by Partner Autocomplete is often useless as it is
quite generic/random e.g. info@company.com. As it is set automatically,
the user might not realize he is going to send an email to that
generic address.
- not returning the email address faclitates some onboarding flows. It
allows us to prompt the user to set his own email address to have a
preview of the emails we are sending.
Note that this information is not lost but rather displayed in the
chatter of the partner record.
However, we do not propagate this modification to other services like
Lead Enrichment or Lead Generation since this kind of email is still
relevant to:
- approach a new prospect
- enrich information and create leads based on the email domain
TaskID 2321864
closes odoo/odoo#56038
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>
Diffstat (limited to 'addons/partner_autocomplete/static')
| -rw-r--r-- | addons/partner_autocomplete/static/src/js/partner_autocomplete_core.js | 2 | ||||
| -rw-r--r-- | addons/partner_autocomplete/static/tests/partner_autocomplete_tests.js | 8 |
2 files changed, 3 insertions, 7 deletions
diff --git a/addons/partner_autocomplete/static/src/js/partner_autocomplete_core.js b/addons/partner_autocomplete/static/src/js/partner_autocomplete_core.js index bc35c58f538..39acfc4c496 100644 --- a/addons/partner_autocomplete/static/src/js/partner_autocomplete_core.js +++ b/addons/partner_autocomplete/static/src/js/partner_autocomplete_core.js @@ -115,7 +115,7 @@ var PartnerAutocompleteMixin = { var self = this; var removeUselessFields = function (company) { - var fields = 'label,description,domain,logo,legal_name,ignored'.split(','); + var fields = 'label,description,domain,logo,legal_name,ignored,email'.split(','); fields.forEach(function (field) { delete company[field]; }); diff --git a/addons/partner_autocomplete/static/tests/partner_autocomplete_tests.js b/addons/partner_autocomplete/static/tests/partner_autocomplete_tests.js index 60fa4679f03..7b07bd99455 100644 --- a/addons/partner_autocomplete/static/tests/partner_autocomplete_tests.js +++ b/addons/partner_autocomplete/static/tests/partner_autocomplete_tests.js @@ -107,7 +107,6 @@ odoo.define('partner_autocomplete.tests', function (require) { city: "Ramillies", zip: "1367", phone: "+1 650-691-3277", - email: "info@odoo.com", vat: "BE0477472701", }; @@ -124,7 +123,6 @@ odoo.define('partner_autocomplete.tests', function (require) { parent_id: {string: "Company", type: "many2one", relation: "res.partner"}, website: {string: "Website", type: "char", searchable: true}, image_1920: {string: "Image", type: "binary", searchable: true}, - email: {string: "Email", type: "char", searchable: true}, phone: {string: "Phone", type: "char", searchable: true}, street: {string: "Street", type: "char", searchable: true}, city: {string: "City", type: "char", searchable: true}, @@ -187,7 +185,7 @@ odoo.define('partner_autocomplete.tests', function (require) { QUnit.test("Partner autocomplete : Company type = Company / Name search", async function (assert) { - assert.expect(18); + assert.expect(17); var fields = this.data['res.partner'].fields; var form = await createView({ View: FormView, @@ -199,7 +197,6 @@ odoo.define('partner_autocomplete.tests', function (require) { '<field name="name" widget="field_partner_autocomplete"/>' + '<field name="website"/>' + '<field name="image_1920" widget="image"/>' + - '<field name="email"/>' + '<field name="phone"/>' + '<field name="street"/>' + '<field name="city"/>' + @@ -261,7 +258,7 @@ odoo.define('partner_autocomplete.tests', function (require) { }); QUnit.test("Partner autocomplete : Company type = Company / VAT search", async function (assert) { - assert.expect(29); + assert.expect(27); var fields = this.data['res.partner'].fields; var form = await createView({ View: FormView, @@ -273,7 +270,6 @@ odoo.define('partner_autocomplete.tests', function (require) { '<field name="name" widget="field_partner_autocomplete"/>' + '<field name="website"/>' + '<field name="image_1920" widget="image"/>' + - '<field name="email"/>' + '<field name="phone"/>' + '<field name="street"/>' + '<field name="city"/>' + |
