[18.0][ADD]l10n_it_edi_related_document: l10n_it_edi extension for multiple related document type management#4749
Conversation
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
26f0293 to
bdcf23f
Compare
matteotognini
changed the title
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
2 times, most recently
from
6a0954b to
d4c7a72
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie della PR!
Come dicevo stamattina in chiamata, secondo me anche questo si potrebbe includere in l10n_it_edi_extension (#4411) perché non vedo grandi differenze tra questa funzionalità e la visualizzazione dell'anteprima, o le altre funzionalità già incluse in quel modulo.
Anche lasciarlo in un modulo a sé ha senso, ma allora potevamo farlo anche per le altre funzionalità 🤷
Quindi per me vanno bene entrambe le soluzioni.
Non ho visto script di migrazione, quindi se uno migra da 16.0 perde tutti i documenti collegati alle fatture?
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
2 times, most recently
from
d691ce1 to
0e22e76
Compare
|
@monen17 Grazie mille per la review! Seguendo le indicazioni e commenti nella task del portale Odoo Italia ho scelto di farlo in un modulo separato. Non ci sarebbero comunque problemi ad aggiungere le funzionalità in l10n_it_edi_extension. |
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
ba46619 to
6cf4ab5
Compare
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie delle modifiche!
Oltre ai punti ancora aperti qui sotto, manca ancora lo script di migrazione.
Ah e ricorda di modificare anche il messaggio del commit visto che il nome del modulo è cambiato.
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
2 times, most recently
from
190129c to
fbc1827
Compare
|
@monen17 Grazie anche per questa review! Ho aggiornato la pr per gestire il collegamento tra le righe di fattura e i documenti collegati, sia in import xml che nel create. |
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Ho fatto solo revisione del codice e secondo me manca solo più qualche piccola modifica; non ho modo di verificare la migrazione ma a occhio mi sembra ok.
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
fbc1827 to
0a90443
Compare
matteotognini
changed the title
|
/ocabot migration l10n_it_edi_related_document |
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
2e03825 to
5a2c2b4
Compare
|
@monen17 puoi re-revisionare? |
| ) | ||
|
|
||
|
|
||
| @openupgrade.migrate() |
There was a problem hiding this comment.
@Borruso puoi aggiornare la review visto che è stata sistemata la migrazione?
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie delle modifiche!
Ho fatto revisione del codice e per me è ok.
Non ho guardato la migrazione, e come ha detto stamattina @eLBati in chiamata potrebbe essere che non venga proprio eseguita essendo questo un nuovo modulo.
Stamattina però abbiamo anche detto che non è necessario migrare i dati per procedere al merge quindi ok così.
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
b084dfb to
1f8ff2e
Compare
|
Ho testato le migrazioni in locale e cosi vengono applicate correttamente all'installazione del modulo |
matteotognini
force-pushed
the
18.0-add-l10n_it_edi_document
branch
from
79bd60b to
58529a4
Compare
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at a51ea89. Thanks a lot for contributing to OCA. ❤️ |
8 participants
Allows import and export of multiple related document types, similar to l10n_it_fatturapa. Also removes the read-only restriction on lineRef. Removes the invoice_line reference from the related document type model, as it is only needed in the report.
Hides l10n_it_edi fields from the invoice view, but keeps them available for import.