OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
22 stars 51 forks source link

[14.0][IMP] l10n_ar_afipws: improves the way in which the connection to afip ws is chosen #72

Closed adrianpaesani closed 1 year ago

adrianpaesani commented 1 year ago

This PR improves the way in wich the connection to AFIP WS is chosen, before this change, it must be configured manually in system parameters, now this is done from the accounting configuration. In addition, sets a base to change all the way AFIP WS are configured in a more simplified way.

OCA-git-bot commented 1 year ago

Hi @ibuioli, @nimarosa, some modules you are maintaining are being modified, check this out!

nimarosa commented 1 year ago

Hola Adrián, gracias por el PR,

Los cambios están bien, pero te pediría que el nombre de los commit sigan las reglas de OCA, ya que es importante luego para las migraciones y otros scripts. En específico:

Te sugiero que hagas un git rebase -i al head, y hagas un squash de commits en uno y luego un force push para meter todos los cambios en un solo commit el cual debes renombrar con la nomenclatura anteriormente.

PD: Siento este el repositorio exclusivo para argentina no hay problema con escribir en español. Aunque los commits si siempre en inglés.

nimarosa commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-72-by-nimarosa-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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