search

OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
291 stars 520 forks source link

[16.0][IMP] payment_redsys: remove website_sale dependency #3777

Closed IJOL closed 3 weeks ago

IJOL commented 1 month ago

@sergio-teruel

pedrobaeza commented 1 month ago

Utiliza por favor un mensaje de commit adecuado: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

ValentinVinagre commented 1 month ago

Reference: https://github.com/OCA/l10n-spain/issues/3751

ValentinVinagre commented 1 month ago

@Jaimermaccione

pedrobaeza commented 1 month ago

Hola, @IJOL el mensaje de commit debe incluir el nombre del módulo. Echa un vistazo a https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

¿Puedes cambiarlo y fusionamos?

HaraldPanten commented 1 month ago

@IJOL Como dice Pedro. Pon un mensaje de commit que siga las guidelines de OCA. Por ejemplo --> [IMP] payment_redsys: remove website_sale dependency.

Y con eso ya lo tiramos adelante. Gracias.

HaraldPanten commented 3 weeks ago

@IJOL Pudiste echarle un vistazo a los comentarios de Pedro?

En caso de que no tengas disponibilidad, vamos a lanzar nosotros el PR.

Gracias.

HaraldPanten commented 3 weeks ago

@pedrobaeza Si lo ves bien, lo tiramos adelante. El mensaje de commit ya tiene el formato/estructura adecuado.

pedrobaeza commented 3 weeks ago

Sí, como no avisó, no sabía que lo había cambiado.

/ocabot merge patch

OCA-git-bot commented 3 weeks ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-3777-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 3 weeks ago

Congratulations, your PR was merged at 913c99d86034894da989f32ca1444aaaa75faffc. Thanks a lot for contributing to OCA. ❤️

HaraldPanten commented 3 weeks ago

Nos encargamos de llevarlo a otras versiones. Gracias por la aportación y las revisiones 👍