search

OCA / l10n-france

France Localization for Odoo
GNU Affero General Public License v3.0
42 stars 118 forks source link

16.0 mig l10n fr ecotaxe #448

Closed mourad-ehm closed 5 months ago

legalsylvain commented 1 year ago

/ocabot migration l10n_fr_ecotaxe

pfranck commented 1 year ago

Any help needed to merge the request ?

pfranck commented 1 year ago

Any solution to be able to merge this request

OCA-git-bot commented 8 months 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). 🤖

gurneyalex commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-448-by-gurneyalex-bump-nobump, awaiting test results.

OCA-git-bot commented 5 months ago

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

bealdav commented 5 months ago

Oops these modules has been hardly modified here

https://github.com/OCA/account-fiscal-rule/pull/428 https://github.com/OCA/account-fiscal-rule/pull/429

Inside Ecotax use tax odoo mechanism

gurneyalex commented 5 months ago

hello @bealdav sorry I had missed the discussion on https://github.com/OCA/l10n-france/issues/483 and the PRs you mention.

I just updated https://github.com/OCA/l10n-france/issues/373 and https://github.com/OCA/l10n-france/issues/487 with the information of the moving the modules.

I'm not sure what the best course is here.

gurneyalex commented 5 months ago

https://github.com/OCA/l10n-france/pull/552 is reverting this

bealdav commented 5 months ago

It's our own fault. We had not a really good communication on it, sorry.