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
[ADD] Pricer tags section added to PoS/pricing #6539
Conversation
4f6deb6
to
1e48a37
Compare
1e48a37
to
1eff662
Compare
1eff662
to
023f603
Compare
3c1d5e4
to
fdedc0b
Compare
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.
Hello Miss Loredana,
Great job! I left some comments, but other than a little punctuation typo, there's nothing that objectively needs to be changed, just some suggestions. You're 100% free to disregard them if you disagree ;)
If you want to discuss in person, let me know and we do it on Thursday!
Putting this on request changes so you can check the comments.
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
384a970
to
c5f1728
Compare
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.
Good to go! :)
c5f1728
to
1aa0a3a
Compare
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.
Two minor suggestions. @robodoo delegate+
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/digital_labels.rst
Outdated
Show resolved
Hide resolved
task-3603496 This PR adds the documentation needed to use Point Of Sale with Pricer electronic price tags. Related PRs: odoo/enterprise#50368 task-3603496
1aa0a3a
to
89b91e8
Compare
@robodoo r+ |
I'm sorry, @LoredanaLrpz. I'm afraid I can't do that. |
@robodoo r+ |
This PR adds the documentation needed to use Point Of Sale with Pricer electronic price tags.
Related PRs:
Pos Pricer module added:
https://github.com/odoo/enterprise/pull/50368
task-3603496