-
-
Notifications
You must be signed in to change notification settings - Fork 603
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
[16.0][MIG] pos_payment_method_cashdro: migration to 16.0 #1161
[16.0][MIG] pos_payment_method_cashdro: migration to 16.0 #1161
Conversation
This module allows to make payments with a Cashdro (https://www.cashdro.com) terminal on the Point of Sale frontend. TT29224
Currently translated at 100.0% (8 of 8 strings) Translation: pos-14.0/pos-14.0-pos_payment_method_cashdro Translate-URL: https://translation.odoo-community.org/projects/pos-14-0/pos-14-0-pos_payment_method_cashdro/it/
Overcome floting point issues: https://stackoverflow.com/a/588014 For expample 2.3 * 100 -> 229.99999999999997 TT42574
Currently translated at 100.0% (8 of 8 strings) Translation: pos-14.0/pos-14.0-pos_payment_method_cashdro Translate-URL: https://translation.odoo-community.org/projects/pos-14-0/pos-14-0-pos_payment_method_cashdro/it/
20cb7b3
to
28df9b0
Compare
/ocabot migration pos_payment_method_cashdro |
f9ec324
to
9bc53b4
Compare
9bc53b4
to
b77ae19
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.
Thanks! 👍
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.
/ocabot merge nobump
On my way to merge this fine PR! |
Congratulations, your PR was merged at 37a27cf. Thanks a lot for contributing to OCA. ❤️ |
@Tecnativa TT45180
please review @CarlosRoca13 @victoralmau
It can be tested with this mock backend: drde5.wiremockapi.cloud