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

[FW][IMP] developer/localization: adapt link to fiscal position file #12586

Closed
wants to merge 1 commit into from

Conversation

fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Mar 21, 2025

Forward-Port-Of: #12189

@robodoo
Copy link
Collaborator

robodoo commented Mar 21, 2025

Pull request status dashboard

@fw-bot
Copy link
Collaborator Author

fw-bot commented Mar 21, 2025

@jco-odoo @Feyensv while this was properly forward-ported, at least one co-dependent PR (odoo/odoo#202928) did not succeed. You will need to fix it before this can be merged.

Both this PR and the others will need to be approved via @robodoo r+ as they are all considered “in conflict”.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@jco-odoo
Copy link
Contributor

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Mar 21, 2025

@jco-odoo @Feyensv linked pull request(s) odoo/odoo#202928 not ready. Linked PRs are not staged until all of them are ready.

robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 22, 2025
This commit adds Canary Island CoA from OCA's l10n_es_igic module.

The initial migration was done using the script from:
`https://github.com/william-andre/transform_coa`

Along that, we also applied few more things:

- use the english translations in `name` and moved the spanish names in
  `name@es`
- added auto-apply on canary's fiscal position based on states
- adapt `-100` taxes on some of the taxes in canary
- refactored the template flow

The template dependencies will be like the following:

```
              es_common (withholding taxes + common accounts)
           /            \
  es_canary_common   es_common_mainland
(taxes + canary accounts)   (taxes)
        |                   |
(canary-assoc/full/pymes)  (assec/full/pymes/coop_pymes)
 same accounts used
as in mainland through
csv loading override
```

Thanks to Binhex (specially Christian).

Task-id: 3354514
Part-of: #202928
Related: odoo/documentation#12586
Signed-off-by: Ruben Gomes (rugo) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 22, 2025
We added the modelo 420 to be able to calculate
the basic tax return for Canary Islands.  (but
no BOE yet)

The report is in 2 columns to distinguish base
amount and tax amount and the casilla numbers
are put at the start of each report line.

Thanks to Binhex (specially Christian).

closes #202928

Task-id: 3354514
Related: odoo/documentation#12586
Signed-off-by: Ruben Gomes (rugo) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
robodoo pushed a commit that referenced this pull request Mar 22, 2025
closes #12586

Forward-port-of: #12189
Related: odoo/odoo#202928
Signed-off-by: Victor Feyens (vfe) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 22, 2025
This commit adds Canary Island CoA from OCA's l10n_es_igic module.

The initial migration was done using the script from:
`https://github.com/william-andre/transform_coa`

Along that, we also applied few more things:

- use the english translations in `name` and moved the spanish names in
  `name@es`
- added auto-apply on canary's fiscal position based on states
- adapt `-100` taxes on some of the taxes in canary
- refactored the template flow

The template dependencies will be like the following:

```
              es_common (withholding taxes + common accounts)
           /            \
  es_canary_common   es_common_mainland
(taxes + canary accounts)   (taxes)
        |                   |
(canary-assoc/full/pymes)  (assec/full/pymes/coop_pymes)
 same accounts used
as in mainland through
csv loading override
```

Thanks to Binhex (specially Christian).

Task-id: 3354514
Part-of: #202928
Related: odoo/documentation#12586
Signed-off-by: Ruben Gomes (rugo) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 22, 2025
We added the modelo 420 to be able to calculate
the basic tax return for Canary Islands.  (but
no BOE yet)

The report is in 2 columns to distinguish base
amount and tax amount and the casilla numbers
are put at the start of each report line.

Thanks to Binhex (specially Christian).

closes #202928

Task-id: 3354514
Related: odoo/documentation#12586
Signed-off-by: Ruben Gomes (rugo) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
robodoo pushed a commit that referenced this pull request Mar 22, 2025
closes #12586

Forward-port-of: #12189
Related: odoo/odoo#202928
Signed-off-by: Victor Feyens (vfe) <[email protected]>
Signed-off-by: Josse Colpaert (jco) <[email protected]>
@robodoo robodoo closed this Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants