-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
[14.0][FIX][l10n_it_fatturapa_in] do not fail import of XML e-invoice on missing/wrong bank account information #4219
base: 14.0
Are you sure you want to change the base?
Conversation
b44ba0e
to
3364ff2
Compare
@TheMule71 Sto facendo il porting della tua pr, pensavo appena finisco di testare tutto di schiacciare tutto nel primo commit per te va bene? |
665b9c0
to
107ad2e
Compare
@tafaRU tu cosa ne pensi che schiaccio tutto in unico commit? cosi poi in caso e' pronta la pr |
Se per @TheMule71 va bene squasha pure, puoi approfittarne per dare un nome più rilevante al commit |
appena mi da conferma procedo subito che in locale sono su questo branch |
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.
rimosso
@@ -843,9 +843,6 @@ def _createPaymentsLine(self, payment, line, partner_id, invoice_id): | |||
) | |||
) | |||
val = { | |||
"payment_term_start": dline.DataRiferimentoTerminiPagamento |
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.
Capisco la prima riga, che è ripetuta sotto, ma payment_bank? è effettivamente inutile?
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.
Si, squasha pure. |
Grazie milleMatteo ***@***.*** Il giorno 20 giu 2024, alle ore 12:29, Marco Colombo ***@***.***> ha scritto:
Se per @TheMule71 va bene squasha pure, puoi approfittarne per dare un nome più rilevante al commit
appena mi da conferma procedo subito che in locale sono su questo branch
Si, squasha pure.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
Openforce Srls Unipersonale | www.openforce.it | 02 21119472 | Via Tiziano,32 20145 Milano
|
107ad2e
to
d95d0a5
Compare
fatto |
ho anche rimesso il messaggio del tuo commit |
la riporto un attimo in draft che testando in locale ho notato un paio di problemi |
3301e5a
to
9f6d8df
Compare
9f6d8df
to
c2c4249
Compare
@TheMule71 ho finito i test e ho schiacciato definitivamente tutto in un commit, fammi sapere se e' tutto chiaro o meno e secondo me ci siamo |
@TheMule71 ci siamo quindi per te? |
@TheMule71 ci siamo? cosi potremmo vedere per mergiare |
@TheMule71 per caso ci siamo per te e possiamo mergiare? |
Porting di #1979