Skip to content

l10n_it_edi_oss#5130

Closed
eLBati wants to merge 2 commits intoOCA:18.0from
eLBati:l10n_it_edi_oss
Closed

l10n_it_edi_oss#5130
eLBati wants to merge 2 commits intoOCA:18.0from
eLBati:l10n_it_edi_oss

Conversation

@eLBati
Copy link
Copy Markdown
Member

@eLBati eLBati commented Feb 26, 2026

No description provided.

@Borruso
Copy link
Copy Markdown
Contributor

Borruso commented Mar 5, 2026

#5023

@eLBati eLBati force-pushed the l10n_it_edi_oss branch from 5f32e28 to 659289c Compare March 6, 2026 14:24
@eLBati
Copy link
Copy Markdown
Member Author

eLBati commented Mar 6, 2026

@Borruso grazie, ho confrontato le 2 PR. Questa sembra avere un approccio più semplice e più integrato , facendo solo degli override e non intervenendo negli XML.
Ho integrato comunque delle cose dalla #5023 , vedi 659289c , mettendoti come autore.
Credi che mi sia perso qualcosa e le modifiche a invoice_it_template.xml siano necessarie? O pensi che sia comunque meglio la #5023 per altri motivi?
Grazie

VBNext
VBNext reviewed Mar 8, 2026
View reviewed changes
l10n_it_edi_oss/constants.py
@@ -0,0 +1,2 @@
OSS_EXEMPT_REASON = "N3.2"
Copy link
Copy Markdown

@VBNext VBNext Mar 8, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Credo sia N7 per chi non è esportatore abituale

Copy link
Copy Markdown
Member Author

@eLBati eLBati Mar 16, 2026

Choose a reason for hiding this comment

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

Sulla 16 però facevamo così

l10n-italy/l10n_it_fatturapa_out_oss/__init__.py

Line 10 in 668de77

"kind_id": env.ref("l10n_it_account_tax_kind.n3_2").id,

Copy link
Copy Markdown

@VBNext VBNext Mar 16, 2026

Choose a reason for hiding this comment

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

Si si, ho visto. Probabilmente è il caso di maggior uso.
Ma perchè portiamo avanti questa e non integriamo l10n_eu_oss con la possibilità di usare N3.2 invece che N7?

@francesco-ooops francesco-ooops added missing issue PRs must be linked to an issue. new module 18.0 labels Mar 13, 2026
@eLBati
Copy link
Copy Markdown
Member Author

eLBati commented Mar 16, 2026

Nuova versione , basata su standard l10n_eu_oss: #5147

@eLBati eLBati closed this Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@VBNext VBNext VBNext left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

18.0 missing issue PRs must be linked to an issue. new module

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@eLBati @Borruso @VBNext @francesco-ooops