search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[FIX] l10n_it_reverse_charge: force amount_currency set in line values #4238

Open odooNextev opened 5 days ago

odooNextev commented 5 days ago

Ho provato a correggere in maniera un po' empirica il problema segnalato nella issue: https://github.com/OCA/l10n-italy/issues/4111

Dopo un po' di test sembra reggere, ma di sicuro necessita di reviews.

odooNextev commented 5 days ago

@Borruso @eLBati @francesco-ooops

eLBati commented 4 days ago

Si sovrappone mica a https://github.com/OCA/l10n-italy/pull/3879 ?

odooNextev commented 4 days ago

@SirAionTech ho aggiunto un test giusto per cominciare, poi bisognerà aggiungere dei check sugli importi e non solo sulla creazione e riconciliazione come adesso. Mi sono fermato perchè non riesco vorrei capire meglio lo scopo di with_supplier_self_invoice nelle posizioni fiscali: mi sembra di capire che nei test è usato per le fatture extra EU, ma secondo i miei colleghi non serve un autofattura addizionale neanche in questo caso.

OCA-git-bot commented 7 hours ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖