Skip to content

[ADD] l10n_it_edi_extension#4858

Merged
OCA-git-bot merged 29 commits into
OCA:18.0from
monen17:18.0-imp-l10n_it_edi_extension-review
Sep 5, 2025
Merged

[ADD] l10n_it_edi_extension#4858
OCA-git-bot merged 29 commits into
OCA:18.0from
monen17:18.0-imp-l10n_it_edi_extension-review

Conversation

@monen17

@monen17 monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor

Sostituisce #4411.

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_fatturapa

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jul 28, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 28, 2025
46 tasks
@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_fatturapa_in

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_fatturapa_out

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_fiscal_payment_term

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_fiscalcode

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_ipa

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_pec

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_rea

@monen17

monen17 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_vat_payability

@monen17 monen17 force-pushed the 18.0-imp-l10n_it_edi_extension-review branch from a046f6b to 5d0b270 Compare July 28, 2025 18:10
@monen17 monen17 marked this pull request as ready for review July 28, 2025 18:11
@monen17 monen17 force-pushed the 18.0-imp-l10n_it_edi_extension-review branch from 5d0b270 to f852f42 Compare July 28, 2025 18:34

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Secondo me OK

@Borruso @odooNextev cosa dite?

@Borruso

Borruso commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

Secondo me OK

@Borruso @odooNextev cosa dite?

per me no

@eLBati

eLBati commented Jul 29, 2025

Copy link
Copy Markdown
Member

Secondo me OK
@Borruso @odooNextev cosa dite?

per me no

cioè?

@Borruso

Borruso commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

vorrei continuare sulla mia

@tafaRU

tafaRU commented Jul 29, 2025

Copy link
Copy Markdown
Member

@monen17 mi aspetterei di vedere il campo l10n_edi_it_create_partner qui ma non è così. Mi sfugge semplicemente qualcosa o c'è un problema nella visualizzazione?

image

Nota: lo screenshot è stato fatto a partire da http://oca-l10n-italy-18-0-pr4858-f852f423dfd6.runboat.odoo-community.org

@monen17

monen17 commented Jul 29, 2025

Copy link
Copy Markdown
Contributor Author

@monen17 mi aspetterei di vedere il campo l10n_edi_it_create_partner qui ma non è così. Mi sfugge semplicemente qualcosa o c'è un problema nella visualizzazione?

image Nota: lo screenshot è stato fatto a partire da http://oca-l10n-italy-18-0-pr4858-f852f423dfd6.runboat.odoo-community.org

Grazie della segnalazione!
Ma è chiaramente qui: image 😆
Ho aggiunto un commit per renderlo più visibile:
image
(da http://oca-l10n-italy-18-0-pr4858-4fbad865bac5.runboat.odoo-community.org/odoo/settings#account)
fammi sapere se ora va meglio

@monen17 monen17 force-pushed the 18.0-imp-l10n_it_edi_extension-review branch from 4fbad86 to 24de1ba Compare July 29, 2025 17:40
Comment thread l10n_it_edi_extension/__init__.py Outdated
Comment on lines +219 to +228
[
(
"account.move.line",
"ftpa_line_number",
),
(
"account.move.line",
"sequence",
),
],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

questa rinomina collide con il modulo account, nel senso che sequence già esiste perché creato da account e quando tenta di rinominarlo va in errore. Questo è quello che sto vedendo su un cliente che sto migrando alla 18.0

Comment thread l10n_it_edi_extension/__init__.py Outdated
Comment on lines +329 to +338
[
(
"res.company",
"fatturapa_tax_representative",
),
(
"res.company",
"l10n_it_tax_representative_partner_id",
),
],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

l10n_it_tax_representative_partner_id esiste già perché definito in l10n_it_edi https://github.com/odoo/odoo/blob/18.0/addons/l10n_it_edi/models/res_company.py#L74 , per cui anche qui va in errore la rinomina

Comment thread l10n_it_edi_extension/__init__.py Outdated
"""
openupgrade.logged_query(env.cr, query)

env.cr.execute("SELECT * FROM fatturapa_activity_progress LIMIT 1")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

se lo scopo di SELECT * FROM fatturapa_activity_progress LIMIT 1 è di verificare se ci sia almeno una riga non credo sia necessario dato che in quel caso la SELECT non ritornerebbe nessun record.
D'altra parte in un caso reale di migrazione mi ha dato problemi perché fatturapa_activity_progress non è detto che esista come tabella, per cui ho modificato la condizione così:

if table_exists(env.cr, 'fatturapa_activity_progress'):
[...]

Stesso discorso per fatturapa_summary_data poco più sotto

@monen17 monen17 force-pushed the 18.0-imp-l10n_it_edi_extension-review branch from 7816697 to d2c7811 Compare August 2, 2025 11:32
@andreampiovesana

Copy link
Copy Markdown
Contributor

merge?


1. Anteprima e Download del file XML:

- Aggiunge un pulsante ("Preview XML") direttamente nel form della fattura.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • Va modificato il formato di stampa.
    stampa attuale:
Screenshot 2025-09-01 alle 11 17 18 Stampa attesa: Screenshot 2025-09-01 alle 11 21 44
  • Manca la possibilità di scelta formato anteprima tra assosoftware -fattura ordinaria - sud tirolo (nella 16 la selezione era in Impostazioni>Contabilità>Fatture elettroniche
Screenshot 2025-09-01 alle 10 35 32 Nella 18 si potrebbe mettere nelle info generali dell'azienda Screenshot 2025-09-01 alle 10 39 01

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • Va modificato il formato di stampa

fixing with monen17#4

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • Manca la possibilità di scelta formato anteprima tra assosoftware -fattura ordinaria - sud tirolo

È stato deciso di lasciare solo assosoftware al momento


- Aggiunge un'opzione per creare i contatti presenti in una fattura elettronica se non esistono in anagrafica tra cui:
- `<CessionarioCommittente>` e `<CedentePrestatore>` invece di scrivere solamente i dettagli nel chatter.
- Creazione di altri contatti presenti in una fattura elettronica se non esistono in anagrafica tra cui:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Non sono campi che serve importare ma esportare.
Manca la possibilità di abbinarli all'anagrafica di clienti e fornitori.
Rimarrebbero quindi dei contatti a se.
ES mi trovo salvato il contatto TEAMSYSTEM perchè il mio fornitore utilizza quel gestionale

@VBNext VBNext Sep 3, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Questo è quello che succede importando i contatti del rappresentante fiscale e del terzo intermediario dalle fatture fornitori. Io eliminerei questa funzionalità, sono contatti che non mi servono nel caso dei fornitori
Screenshot 2025-09-03 alle 14 23 10

@VBNext VBNext Sep 3, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

La creazione di Cedente Prestatore funziona correttamente

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Io eliminerei questa funzionalità, sono contatti che non mi servono nel caso dei fornitori

Rimuovendo con monen17#4

- `<CessionarioCommittente>` e `<CedentePrestatore>` invece di scrivere solamente i dettagli nel chatter.
- Creazione di altri contatti presenti in una fattura elettronica se non esistono in anagrafica tra cui:
- `<RappresentanteFiscale>`
- `<TerzoIntermediarioOSoggettoEmittente>`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ho la possibilità di impostare il terzo intermediario nella company, ma il dato non viene riportato in fattura elettronica
Screenshot 2025-09-01 alle 11 44 23
Screenshot 2025-09-01 alle 11 29 40

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

non si può aggiungere questa nuova funzionalità in una pr successiva?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Non è nuova, nella 16 funzionava correttamente

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment on lines +132 to +136
- Creation of article codes during import.
These codes are used to uniquely identify products/services according to different coding standards.
In the XML file, these codes are in the `<CodiceArticolo>` node which can contain:
- `<CodiceTipo>`: identifies the type of coding used
- `<CodiceValore>`: the actual code value

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Da rendere facoltativa, o limitarla ad alcuni fornitori a scelta.
Il rischio è di trovarsi l'anagrafica prodotti popolata da prodotti che acquisto solo 1 volta (es. se compro spesso da amazon)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

L'importazione non mi risulta crei nuovi prodotti, crea solo l10n_it_edi.article_code, che trovi in "E-invoice details"

@eLBati

eLBati commented Sep 5, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-4858-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c85f092 into OCA:18.0 Sep 5, 2025
5 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

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.