Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[18.0][ADD] l10n_fr_account_oca: OCA fork of l10n_fr_account #628

Open
wants to merge 2 commits into
base: 18.0
Choose a base branch
from

Conversation

alexis-via
Copy link
Contributor

Since v14, OCA/l10n-france has a module l10n_fr_oca which is a fork of l10n_fr.
In v18, the chart of accounts was completely reorganized and l10n_fr has been split between l10n_fr and l10n_fr_account. So the best solution was to create a new fork of l10n_fr_account to l10n_fr_account_oca.

Since v14, OCA/l10n-france has a module l10n_fr_oca which is a fork of
l10n_fr. In v18, the chart of accounts was completely reorganized and
l10n_fr has been split between l10n_fr and l10n_fr_account. So the best
solution was to create a new fork of l10n_fr_account to l10n_fr_account_oca.
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @alexis-via.

thanks for porting this module.

A format remark inline.

Otherwise, LGTM.

Thanks !

@@ -0,0 +1,675 @@
id,name,code,account_type,tag_ids,reconcile
pcg_1011,Capital souscrit - non appelé,101100,equity,,False
Copy link
Contributor

@legalsylvain legalsylvain Feb 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi. could you avoid to change the format of the file regarding the original one ?
(https://raw.githubusercontent.com/odoo/odoo/refs/heads/18.0/addons/l10n_fr_account/data/template/account.account-fr.csv)
I mean :

  • do not change delimiter
  • do not remove column

So it is allways easy to compare / meld both files. (the original ones, and the forked one).

thanks !

@florian-dacosta
Copy link
Contributor

Fix pre-commit + differenciate native and OCA CoA in the configuration list

akretion#13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants