-
Notifications
You must be signed in to change notification settings - Fork 10k
[IMP] pos/terminals:update Payment terminals doc #14384
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
Conversation
bcab0cd
to
c228700
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.
Thank you for your work @emmi-odoo! Feel free to ask if some of my comments/suggestions aren't clear 😉
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
2e71dc1
to
6dc0778
Compare
The changes and suggestions have been implemented. However, I'm not satisfied with the sentence in the tip. Can it be improved? |
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/payment_methods/terminals.rst
Outdated
Show resolved
Hide resolved
task-4946909
6dc0778
to
b65b1ff
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.
Thank you for the changes @emmi-odoo! LGTM 🙂
@robodoo r+
task-4946909