[18.0][MIG] l10n_it_pos_fiscalcode: migration to v18#5006
Conversation
|
/ocabot migration l10n_it_pos_fiscalcode |
|
/ocabot rebase |
|
@monen17 The rebase process failed, because command |
@HekkiMelody potresti fare un rebase please? |
The module adds fiscal code field to partner form within point of sale interface, allowing to view and edit it [UPD] Update l10n_it_pos_fiscalcode.pot [UPD] README.rst
Currently translated at 100.0% (2 of 2 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_pos_fiscalcode Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_pos_fiscalcode/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_pos_fiscalcode Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_pos_fiscalcode/
Currently translated at 100.0% (2 of 2 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_pos_fiscalcode Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_pos_fiscalcode/it/
HekkiMelody
force-pushed
the
18.0-mig-pos-fiscalcode
branch
from
003fd4c to
2334a2b
Compare
|
@monen17 Fatto! Grazie |
There was a problem hiding this comment.
Grazie della PR!
Ho fatto revisione del codice e qualche prova, a quanto vedo il modulo potrebbe servire solo per la ricerca tramite codice fiscale, ma magari mi sono perso qualcosa, fammi sapere cosa ne pensi!
Oltre ai commenti qui sotto, potresti schiacciare i commit dei bot come indicato in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate?
Attualmente ce ne sono diversi:
|
|
||
| <t | ||
| t-name="PartnerDetailsEdit" | ||
| t-inherit="point_of_sale.PartnerDetailsEdit" |
There was a problem hiding this comment.
Non ho trovato questa vista nel core, ho il dubbio che non esista più in 18.0, potresti verificare?
| <div class="partner-detail"> | ||
| <span class="label">F.C.</span> | ||
| <input | ||
| class="detail l10n_it_codice_fiscale" | ||
| name="l10n_it_codice_fiscale" | ||
| t-on-change="captureChange" | ||
| t-att-value="props.partner.l10n_it_codice_fiscale || ''" | ||
| /> | ||
| </div> |
There was a problem hiding this comment.
Ho provato in una 18.0 senza questa PR e il cliente viene mostrato con lo stesso form del backend:
Potresti chiarire a cosa servono queste modifiche?
Magari sono in una vista che non so come raggiungere.
| "installable": True, | ||
| "depends": [ | ||
| "point_of_sale", | ||
| "l10n_it_edi_extension", |
There was a problem hiding this comment.
| "l10n_it_edi_extension", | |
| "l10n_it_edi", |
Il campo del codice fiscale è definito da l10n_it_edi (https://github.com/odoo/odoo/blob/0b54f8463200a840a8fde17ceed2c928a3249b68/addons/l10n_it_edi/models/res_partner.py#L15) quindi potremmo dipendere direttamente da quel modulo se non ci serve l10n_it_edi_extension, cosa ne pensi?
| _inherit = "pos.session" | ||
|
|
||
| def _loader_params_res_partner(self): | ||
| result = super()._loader_params_res_partner() |
There was a problem hiding this comment.
Questo metodo non mi pare esista in super, puoi verificare?
Ho fatto una piccola ricerca e temo che se vogliamo poter cercare per codice fiscale dovremo in qualche modo inserirci lato JS in https://github.com/odoo/odoo/blob/0b54f8463200a840a8fde17ceed2c928a3249b68/addons/point_of_sale/static/src/app/screens/partner_list/partner_list.js#L148.
|
Prendo in carico in #5146. |
11 participants
Adattato le dipendenze e rinominato i campi rispetto alla v16