[16.0][MIG] payment redsys: Migration to 16.0 #3125
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Corregido error en caso de que refence no este informado, al renderizar la plantilla del boton de pago. * Corregido error que no permite pagar por transferencia al instalar el modulo payment_redsys * Añadidas descripciones de los productos de la cesta a la clave 'DS_Merchant_ProductDescription'.
* Make quotation sending optional when a transaction ends without error website_sale module already sends an e-mail with the confirmed order, so it May not be necessary to send another e-mail with the same order unconfirmed. * Add more Redsys frontend languages * Remove options tab
* Send quotation e-mail invoking force_quotation_send() instead of action_quotation_send() + send_mail() * The function send_mail() does not respect the context language so the email is always sent in English. * It's not necessary to create the function send_mail() to sent an email when exists the method force_quotation_send(). * Add detailed error info
…r payment + Default values for urk_KO and url_OK + option to print print sale report. Delete url_ok and url_ko fields + Redondeo de merchant_amount
The discount is shown on the order summary if the current user is a member of sale.group_discount_per_so_line group OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex
[IMP] payment_redsys: Allow partial payment by percent
Includes to allow show order for unauthenticated users
When we have an order sequence of length six (SO123456), the signature process fails with the following message: Server Error: We are not able to redirect you to the payment form. 'str' object has no attribute 'decode'
Currently translated at 71.6% (48 of 67 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-payment_redsys/es/ Translated using Weblate (Spanish) Currently translated at 100.0% (67 of 67 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-payment_redsys/es/ Translated using Weblate (Spanish) Currently translated at 100.0% (67 of 67 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-payment_redsys/es/
Currently translated at 100.0% (47 of 47 strings) Translation: l10n-spain-13.0/l10n-spain-13.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-payment_redsys/es/
Currently translated at 97.9% (46 of 47 strings) Translation: l10n-spain-13.0/l10n-spain-13.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-payment_redsys/ca/
…jecutar post procesado al recibir el formulario POST de Redsys + Avoid execute method por non Redsys acquirer
Currently translated at 100.0% (46 of 46 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-payment_redsys Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-payment_redsys/ca/
- JSON request: For a JSON request there's no website attribute. This is the case for payment links. We try to fallback to context if it has the `website_id` in it. Otherwise, `web.base.url` will be our last resource as usual. TT34154 - wrong reference TT34312
pycrypto is unmaintained since several ago, and in fact pycryptodome was the library being used in previous version. The API we are using is compatible between both libraries though. There's no problem for already deployed systems, as the check on the external dependencies is only done at install. We took the occasion to remove the try mechanism that is not needed now, although the linter emits a warning.
We're sending the string template without parameters to the method _set_transcation_error() so no info is shown. TT41589
Rferri44-S73
changed the title
Rferri44-S73
force-pushed
the
16.0-mig-payment_redsys
branch
from
a21fae9
to
d022e58
Compare
Rferri44-S73
force-pushed
the
16.0-mig-payment_redsys
branch
from
d022e58
to
7d9154a
Compare
|
/ocabot migration payment_redsys |
antonyht27
approved these changes
Jul 25, 2023
sergio-teruel
approved these changes
Jul 25, 2023
|
This PR has the |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 385e0b5. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Standard migration to 16.0