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

@lorenzoallegrucci
Copy link

@monen17 @lorenzoallegrucci controlla PR monen17#3 per errori in file di migrazione

nessun errore

@monen17
Copy link
Contributor Author

monen17 commented Aug 2, 2025

@monen17 @lorenzoallegrucci controlla PR monen17#3 per errori in file di migrazione

Grazie, mergiata e inclusa nel commit dove vengono aggiunti gli script

Borruso and others added 23 commits August 2, 2025 13:31
Allow the user to import e-bills with no lines, one line for each tax rate, or all lines (default).
It was removed when migrating this wizard to 18.0, when only domain had
to be removed because no more supported in onchange.

Support has been removed in
odoo/odoo@024fadf.
There is no need to update a partner that has just been created
Also align its configuration in the README to what it actually does
@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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants