OCA / l10n-spain

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

[13.0][ADD] l10n_es_extra_data: nuevos impuestos no soportados por Odoo Oficial #3733

Closed acysos closed 1 month ago

acysos commented 2 months ago

Hola,

Añade el módulo l10n_es_extra_data con los nuevos IVAs 2% u 7,5% y los recargos del 0,26% y 1%. Cuando este aprobado hacemos un cherry pick a 14.0 y metemos otro PR con las modificaciones de los modelos de AEAT.

Saludos

acysos commented 2 months ago

Creo que los test fallan por algo ajeno al módulo:

++ oca_list_external_dependencies deb could not obtain odoo.addons.path using python not found in addons path: account_asset_management,account_banking_sepa_direct_debit,account_check_printing_report_base,account_fiscal_position_partner_type,account_invoice_refund_link,account_payment_partner,account_payment_promissory_note,account_tax_balance,base,base_bank_from_iban,base_iso3166,base_location_geonames_import,delivery_package_number,delivery_price_method,delivery_state,edi_account,intrastat_product,l10n_eu_oss,mis_builder,pos_order_mgmt,queue_job,report_qweb_parameter,report_xlsx,report_xml Aborted.

Saludos

pedrobaeza commented 2 months ago

@sbidoul do you know what is the problem in CI/runboat?

sbidoul commented 2 months ago

I had forgotten about that. I'll investigate.

AntoniRomera commented 2 months ago

Any news on this issue?

acysos commented 2 months ago

Ok, I wait to solve the runbot error. The module works in 14.0. When the PR is merged, the cherry pick is direct.

Thank you

Tardo commented 1 month ago

Any news? can help with something?

pedrobaeza commented 1 month ago

@sbidoul gentle reminder

sbidoul commented 1 month ago

Hi there,

I debugged this by running this command from the l10n-spain repo root:

docker run -it --rm -v .:/_build ghcr.io/oca/oca-ci/py3.6-odoo13.0:latest bash, then running the commands in the test GitHub workflow. The first command is install_oca_addons which fails, on oca_list_external_dependencies deb, with a unhelpful error message.

I then tried python -c 'import odoo.addons' which gave an error about lxml_html_clean that is missing. pip install lxml_html_clean did not help, and that is because Odoo 13 is definitely not compatible with this recent lxml.

pip install "lxml<5.2" does help: can you try adding that to test-requirements.txt?

The root cause is likely a dependency of this repo wanting a more recent lxml than the one in Odoo 13's requirements.txt. And since there is no upper bound for lxml nowhere, it pulls one that is not compatible with Odoo.

OCA-git-bot commented 1 month ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 13.0-ocabot-merge-pr-3733-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 1 month ago

It looks like something changed on 13.0 in the meantime. Let me try again (no action is required from you). Prepared branch 13.0-ocabot-merge-pr-3733-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 1 month ago

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