diff --git a/pos_payment_method_cashdro_fix/README.rst b/pos_payment_method_cashdro_fix/README.rst index 74ef554..d636113 100644 --- a/pos_payment_method_cashdro_fix/README.rst +++ b/pos_payment_method_cashdro_fix/README.rst @@ -51,10 +51,12 @@ To check the three fixes on a PoS with a CashDro terminal: due amount (fix 2). #. The drawer must start the operation and ask for the amount, instead of the *"An error occurred while connecting to the cashdro"* dialog (fixes 1 and 3). -#. With no CashDro at hand, the browser console tells fix 1 from fix 3 apart: - without this module the failure is a ``TypeError: Cannot read properties of - undefined (reading 'payment_method')``; with it, a network error against the - configured host. + +These three fixes are not enough on their own: the payment flow of the base +module is broken on 18.0 for reasons of its own, and it hides every failure — +these ones included — behind that same dialog, without logging anything. +Install ``pos_payment_method_cashdro_flow_fix`` as well; its ``USAGE`` explains +how to read what actually went wrong. The Python side is covered by the tests. The base module lives outside this repo, so the OCA sources have to be on the addons path (mounted by @@ -76,7 +78,7 @@ the credential fields that are missing from the loader, and the JavaScript patches are idempotent with respect to the fixed upstream code. The JavaScript side can't be covered by automated tests, as it needs a CashDro -terminal answering on the network. +terminal answering on the network. See ``USAGE`` for the manual check. Contributors ============ diff --git a/pos_payment_method_cashdro_fix/readme/USAGE.rst b/pos_payment_method_cashdro_fix/readme/USAGE.rst index 2b6d928..ee98fe0 100644 --- a/pos_payment_method_cashdro_fix/readme/USAGE.rst +++ b/pos_payment_method_cashdro_fix/readme/USAGE.rst @@ -7,10 +7,12 @@ To check the three fixes on a PoS with a CashDro terminal: due amount (fix 2). #. The drawer must start the operation and ask for the amount, instead of the *"An error occurred while connecting to the cashdro"* dialog (fixes 1 and 3). -#. With no CashDro at hand, the browser console tells fix 1 from fix 3 apart: - without this module the failure is a ``TypeError: Cannot read properties of - undefined (reading 'payment_method')``; with it, a network error against the - configured host. + +These three fixes are not enough on their own: the payment flow of the base +module is broken on 18.0 for reasons of its own, and it hides every failure — +these ones included — behind that same dialog, without logging anything. +Install ``pos_payment_method_cashdro_flow_fix`` as well; its ``USAGE`` explains +how to read what actually went wrong. The Python side is covered by the tests. The base module lives outside this repo, so the OCA sources have to be on the addons path (mounted by