diff options
| author | fedegobea <65693362+fedegobea@users.noreply.github.com> | 2020-12-12 23:19:20 +0000 |
|---|---|---|
| committer | Joseph Caburnay <jcb@odoo.com> | 2021-01-05 12:02:06 +0000 |
| commit | 94a735ec6a9caa334eca6c4d7a4ca89bf3b5eaf9 (patch) | |
| tree | e844784a2a508d80b4ed43403a359f1d8c6ca541 /addons | |
| parent | b7400c0868e64c077989ccdac62b20c0e862ff5b (diff) | |
[FIX] point_of_sale: editing customers always asking to change name
Hi i detected an issue on POS, to reproduce just get into a customer on the POS UI, change something else than the name, then try to save changes, this trigger the if in line 47, making the user to mandatory edit the name field then save again and it works.
The only change on the file is on line 47 where I check if the name is already on the prop so that it works as intended.
Its a mild UX issue with a very easy fix. hope you can upload it soon to the repo so that users on Odoo SaaS or users with no code skills dont have this Issue anymore-
Best regards
closes odoo/odoo#63281
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/point_of_sale/static/src/js/Screens/ClientListScreen/ClientDetailsEdit.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/addons/point_of_sale/static/src/js/Screens/ClientListScreen/ClientDetailsEdit.js b/addons/point_of_sale/static/src/js/Screens/ClientListScreen/ClientDetailsEdit.js index c50ea05f991..1ac4cf05077 100644 --- a/addons/point_of_sale/static/src/js/Screens/ClientListScreen/ClientDetailsEdit.js +++ b/addons/point_of_sale/static/src/js/Screens/ClientListScreen/ClientDetailsEdit.js @@ -44,7 +44,8 @@ odoo.define('point_of_sale.ClientDetailsEdit', function(require) { processedChanges[key] = value; } } - if (!processedChanges.name) { + if ((!this.props.partner.name && !processedChanges.name) || + processedChanges.name === '' ){ return this.showPopup('ErrorPopup', { title: _('A Customer Name Is Required'), }); |
