-
-
Notifications
You must be signed in to change notification settings - Fork 634
[17.0] [ADD] pos_customer_history_color #1375
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 17.0
Are you sure you want to change the base?
[17.0] [ADD] pos_customer_history_color #1375
Conversation
- Works out-of-the-box with default Odoo POS. | ||
- No changes to backend models. | ||
|
||
## Compatibility |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section is useless, as it is the norm for all the module.
- Faster recognition of frequently purchased items. | ||
- Improved upselling or cross-selling opportunities. | ||
|
||
## Setup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section can be removed, as it is classic installation
|
||
No additional configuration required. | ||
|
||
## Technical Notes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section can be removed.
|
||
## How It Works | ||
|
||
When a product is added to a POS order: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you move this section in a dedicated USAGE.md file ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor remark.
Thanks for sharing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected !
@parvezqureshi can you please apply Sylvain's reviewing ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
except @legalsylvain 's minor remarks in readme
25d5d03
to
ec21788
Compare
The changes have been applied as requested. |
This PR has the |
No description provided.