search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[FIX]l10n_it_financial_statements_report: Syntax compatibility #4178

Closed FrancescoGermani closed 2 weeks ago

FrancescoGermani commented 1 month ago

Report XLSX: the syntax "if isinstance(value, int | float)" can be used on Python 3.10 and above versions. 16.0 code should be compatible with Python>=3.7 tho.

OCA-git-bot commented 3 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 3 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

SirAionTech commented 2 weeks ago

/ocabot merge patch

OCA-git-bot commented 2 weeks ago

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

OCA-git-bot commented 2 weeks ago

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

francesco-ooops commented 2 weeks ago

@SirAionTech non serve per la 14 giusto?

SirAionTech commented 2 weeks ago

@SirAionTech non serve per la 14 giusto?

Giusto, vedi https://github.com/OCA/l10n-italy/issues/4179#issuecomment-2168151289