14.0 mig l10n it fatturapa in purchase #2229
Conversation
fredzamoabg
force-pushed
the
14.0-mig-l10n_it_fatturapa_in_purchase
branch
4 times, most recently
from
0bc374b
to
226e5fb
Compare
fredzamoabg
force-pushed
the
14.0-mig-l10n_it_fatturapa_in_purchase
branch
from
226e5fb
to
16556ff
Compare
fredzamoabg
force-pushed
the
14.0-mig-l10n_it_fatturapa_in_purchase
branch
from
16556ff
to
61a72c9
Compare
fredzamoabg
force-pushed
the
14.0-mig-l10n_it_fatturapa_in_purchase
branch
from
61a72c9
to
592fb75
Compare
| "product_qty", | ||
| "product_uom", | ||
| "invoice_lines.quantity", | ||
| "invoice_lines.uom_id", | ||
| "invoice_lines.product_uom_id", | ||
| ) | ||
| def _compute_to_invoice(self): | ||
| precision = self.env["decimal.precision"].precision_get( |
There was a problem hiding this comment.
Qui sotto puoi usare line.qty_to_invoice invece di ricalcolarlo?
Oppure puoi includere il calcolo del nuovo campo to_invoice dentro il metodo _compute_qty_invoiced che calcola entrambe le quantità che ti servono.
| /> | ||
| <record id="view_invoice_form_fatturapa_po_line" model="ir.ui.view"> | ||
| <field name="name">view_invoice_form_fatturapa_po_line</field> | ||
| <field name="model">account.move</field> |
There was a problem hiding this comment.
Come mai hai rimosso la modifica dal form della riga fattura?
There was a problem hiding this comment.
questa deriva dalla modifica in l10n_it_fatturapa (eb32865#diff-1f57cc2debafd446277a8fdffc044c732422dee3e33bd69d6e07f78be1f1812eR15)
There was a problem hiding this comment.
Recuperando la storia della modifica eb32865#diff-1f57cc2debafd446277a8fdffc044c732422dee3e33bd69d6e07f78be1f1812eR15 ho trovato anche la motivazione per cui non ha senso visualizzare questi campi nelle viste di account.move.line:
Alla fine la vista account.view_move_line_form viene usata per i movimenti contabili, si anche non visualizzare i documenti correlati, perché riguardano l'XML della fattura.
(da #1984 (comment))
Quindi va bene lasciare così 👍🏻
|
@tafaRU your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2229-by-tafaRU-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@tafaRU your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2229-by-tafaRU-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@sbidoul merge failed due to https://github.com/OCA/l10n-italy/runs/4331912624?check_suite_focus=true#step:4:161 Could you please check it? |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@tafaRU your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2229-by-tafaRU-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@tafaRU That is because the setup.py is missing for l10n_it_fatturapa_in_purchase. But what I don't understand is why the pre-commit check does not detect it. |
|
@sbidoul, thanks for your help! Do you think it could be useful open a issue? On MQT? |
|
@fredzamoabg riesegui per favore |
fredzamoabg
force-pushed
the
14.0-mig-l10n_it_fatturapa_in_purchase
branch
from
2bbf09f
to
093e876
Compare
|
@tafaRU issue created OCA/oca-addons-repo-template#93 |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 9760ea7. Thanks a lot for contributing to OCA. ❤️ |
migrazione modulo l10n_it_fatturapa_in_purchase dalla version 12.0
--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing