pos icon indicating copy to clipboard operation
pos copied to clipboard

[ADD] pos_order_zip

Open SMaciasOSI opened this issue 1 year ago • 5 comments

SMaciasOSI avatar Aug 18 '23 22:08 SMaciasOSI

@max3903 @dreispt @AmmarOfficewalaSerpentCS

SMaciasOSI avatar Aug 18 '23 22:08 SMaciasOSI

@fcvalgar please review 😄

MT-4562

@moduon

rafaelbn avatar Jan 22 '24 04:01 rafaelbn

@SMaciasOSI can you rebase , please?

fcvalgar avatar Jan 22 '24 08:01 fcvalgar

Hello @SMaciasOSI @dreispt @max3903, could you please check and answer comments from other contributors please? 🙏🏼 😄

  • [ ] Make it optional or at least the option to make it optional. As said, not all customers will fill the zip and shouldn't be mandatory

It is very blocking that the validation is not optional because in some cases we will not have the data. Customers who want privacy for example.

  • [ ] Validate the ZIP, to be sure it's not fake

Entering a data that is not real will alter the sales reports.

  • [ ] Depends of base_location: https://github.com/OCA/pos/pull/1045#pullrequestreview-1840781823

  • [ ] Make possible to group by in reports https://github.com/odoo/odoo/pull/121594

Let us know please.

If you agree, maybe we could improve it. But I would like to know if you are already using un production.

Thank you! ❤️

rafaelbn avatar Feb 10 '24 17:02 rafaelbn

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jun 16 '24 12:06 github-actions[bot]