Skip to content

Conversation

monen17
Copy link
Contributor

@monen17 monen17 commented Jul 28, 2025

Sostituisce #4411.

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/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
41 tasks
@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_fatturapa_in

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_fatturapa_out

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_fiscal_payment_term

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_fiscalcode

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_ipa

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_pec

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/ocabot migration l10n_it_rea

@monen17
Copy link
Contributor Author

monen17 commented Jul 28, 2025

/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
Copy link
Member

@eLBati eLBati left a comment

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
Copy link
Contributor

Borruso commented Jul 29, 2025

Secondo me OK

@Borruso @odooNextev cosa dite?

per me no

@eLBati
Copy link
Member

eLBati commented Jul 29, 2025

Secondo me OK
@Borruso @odooNextev cosa dite?

per me no

cioè?

@Borruso
Copy link
Contributor

Borruso commented Jul 29, 2025

vorrei continuare sulla mia

@tafaRU
Copy link
Member

tafaRU commented Jul 29, 2025

@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
Copy link
Contributor Author

monen17 commented Jul 29, 2025

@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 on lines 219 to 228
[
(
"account.move.line",
"ftpa_line_number",
),
(
"account.move.line",
"sequence",
),
],

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 on lines 329 to 338
[
(
"res.company",
"fatturapa_tax_representative",
),
(
"res.company",
"l10n_it_tax_representative_partner_id",
),
],

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

"""
openupgrade.logged_query(env.cr, query)

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

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
Contributor

merge?


1. Anteprima e Download del file XML:

- Aggiunge un pulsante ("Preview XML") direttamente nel form della fattura.
Copy link
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
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
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
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

Copy link

@VBNext VBNext Sep 3, 2025

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

Copy link

@VBNext VBNext Sep 3, 2025

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
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
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
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
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
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

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
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
Copy link
Member

eLBati commented Sep 5, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
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
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.