Skip to content
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

[FIX] adyen_platforms: Routes versioning #61236

Open
wants to merge 6 commits into
base: 14.0
Choose a base branch
from

Conversation

aprieels
Copy link
Contributor

@aprieels aprieels commented Nov 3, 2020

Versioning of routes on the proxy

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses seen 🙂 labels Nov 3, 2020
@C3POdoo C3POdoo added the RD research & development, internal work label Nov 3, 2020
@aprieels aprieels force-pushed the 14.0-adyen-platforms-versioning-anp branch from cef373e to 86bc213 Compare January 8, 2021 13:57
Versioning of routes on the proxy
When the customre inputs incorrect card information, an error
notification is sent but the payment is still going, we shouldn't
cancel the transaction in Odoo.
Payouts can have a balance in multiple currencies but bank account can
only have a single currency. Payouts shouldn't be uniquely linked to
bank accounts.
If we give no bank account ID, Adyen will automatically choose a
default one with the correct currency.
The field should be completely removed in master.
We changed the pricing of Odoo Payments so we need to update the
creation confirmation view to show the new pricing of transactions.

Since the pricing flow changed a bit on the proxy, we also need to
update the format of the data of the payment request.
Introduce a test mode for Odoo Payments. No account is required, the
payment will just go to the liable account of the test platform.
@lathama
Copy link
Contributor

lathama commented Apr 15, 2024

@aprieels knowing that 14 is no longer updated, is this PR still needed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants