-
-
Notifications
You must be signed in to change notification settings - Fork 319
[ADD] l10n_it_edi_extension #4858
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 18.0
Are you sure you want to change the base?
Conversation
/ocabot migration l10n_it_fatturapa |
/ocabot migration l10n_it_fatturapa_in |
/ocabot migration l10n_it_fatturapa_out |
/ocabot migration l10n_it_fiscal_payment_term |
/ocabot migration l10n_it_fiscalcode |
/ocabot migration l10n_it_ipa |
/ocabot migration l10n_it_pec |
/ocabot migration l10n_it_rea |
/ocabot migration l10n_it_vat_payability |
a046f6b
to
5d0b270
Compare
5d0b270
to
f852f42
Compare
There was a problem hiding this 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?
per me no |
cioè? |
vorrei continuare sulla mia |
@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? ![]() Nota: lo screenshot è stato fatto a partire da http://oca-l10n-italy-18-0-pr4858-f852f423dfd6.runboat.odoo-community.org |
Grazie della segnalazione! |
4fbad86
to
24de1ba
Compare
l10n_it_edi_extension/__init__.py
Outdated
[ | ||
( | ||
"account.move.line", | ||
"ftpa_line_number", | ||
), | ||
( | ||
"account.move.line", | ||
"sequence", | ||
), | ||
], |
There was a problem hiding this comment.
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
l10n_it_edi_extension/__init__.py
Outdated
[ | ||
( | ||
"res.company", | ||
"fatturapa_tax_representative", | ||
), | ||
( | ||
"res.company", | ||
"l10n_it_tax_representative_partner_id", | ||
), | ||
], |
There was a problem hiding this comment.
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
l10n_it_edi_extension/__init__.py
Outdated
""" | ||
openupgrade.logged_query(env.cr, query) | ||
|
||
env.cr.execute("SELECT * FROM fatturapa_activity_progress LIMIT 1") |
There was a problem hiding this comment.
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
nessun errore |
Grazie, mergiata e inclusa nel commit dove vengono aggiunti gli script |
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.
Co-authored-by: Simone Rubino <[email protected]> Co-authored-by: LorenzoC0 <[email protected]>
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
7816697
to
d2c7811
Compare
merge? |
Sostituisce #4411.