l10n-italy
l10n-italy copied to clipboard
OCA
[14.0][ADD] l10n_it_fatturapa_out: edit invoice sent SDI
Risolve https://github.com/OCA/l10n-italy/issues/2731 per la 14
-- 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
Ciao Marcelo, il primo punto c'è un gruppo che è abilitato al cambio. per la modifica del cliente indago il tool controlla la Piva . per il terzo punto è stata discussa e scelta questa modalità nel venerdi .
Ritengo controproducente per tutti consentire modifiche di questo tipo. Ad esempio sono riuscito a cambiare il Cliente nella fattura e riconfermala mantenendo l'xml inalterato. Se dobbiamo coprire questa funzionalità, farei un wizard con il singolo campo da modificare, dove siamo certi che non risulta presente sul xml e non crea problemi alla fattura, senza dover portare in bozza il documento.
ho provato il cambio cliente . il controllo funziona mi ha bloccato . il sistema controlla la Piva se cambi partner con stessa piva ti è concesso.
se puoi descrivi meglio @marcelofrare FYI @stevech091
Per i test che falliscono, probabilmente l'oggetto request è vuoto / non inizializzato. Probabilmente vale lo stesso per operazioni automatiche, tutto ciò che non segue un'azione da parte dell'utente.
Nota: il motivo di tale controllo è che l'auto post della fattura va fatto solo per le write() chiamate da JS direttamente, per es. col pulsante di Save (per la 16.0 vedremo).
Esiste la possibilità che delle write() vengano chiamate da python (da altri moduli) per es. in button_draft(), che romperebbe la logica del codice qui.
buon pomeriggio a tutti, stavo testando la PR, ma non mi è chiaro una passaggio:
- fattura validata e file XML generato, devo riportarla a bozza per poter fare le modifiche Al riporta a draft la fattura ATTESO viene riportata a bozza la fattura anche se c'è XML OSSERVATO Invoice INV/2023/06/00xx has XML and can't be reset to draft. Delete the XML before.
C'è qualche passaggio che va fatto su qualche vista e che mi sfugge? grazie!
C'è qualche passaggio che va fatto su qualche vista e che mi sfugge? grazie!
C'è un gruppo apposito per portarlo in draft. Edit Invoice Sent SDI
C'è qualche passaggio che va fatto su qualche vista e che mi sfugge? grazie!
C'è un gruppo apposito per portarlo in draft.
Edit Invoice Sent SDI
grazie mille @Borruso
C'è qualche passaggio che va fatto su qualche vista e che mi sfugge? grazie!
C'è un gruppo apposito per portarlo in draft.
Edit Invoice Sent SDI
Si potrebbe documentare nel README o menzionare nel messaggio di errore, cosa ne pensi? Altrimenti l'utente come fa a sapere che c'è questa possibilità?
Si potrebbe documentare nel README o menzionare nel messaggio di errore, cosa ne pensi? Altrimenti l'utente come fa a sapere che c'è questa possibilità?
hai ragione bisogna aggiornare il readme
Le modifiche apportate mi sembrano nel complesso buone. Oltre ai commenti/suggerimenti che trovi nella mia review sarei per implementare alcuni test automatici che coprano la nuova funzionalità.
Grazie!
aggiunto test
Le modifiche apportate mi sembrano nel complesso buone. Oltre ai commenti/suggerimenti che trovi nella mia review sarei per implementare alcuni test automatici che coprano la nuova funzionalità.
Grazie!
@tafaRU aggiunto i test
ciao io ho fatto dei test. Con le fatture di vendita tutto ok, mentre con fatture fornitore estero che generano autofattura non mi fa mai confermare. anche se metto solo in bozza e poi confermo genera un errore
Ho provato a eliminare xml originale e confermala per confrontare i due xml a mano e sono identici
Qualcuno ha provato?
ciao io ho fatto dei test. Con le fatture di vendita tutto ok, mentre con fatture fornitore estero che generano autofattura non mi fa mai confermare. anche se metto solo in bozza e poi confermo genera un errore Ho provato a eliminare xml originale e confermala per confrontare i due xml a mano e sono identici
Qualcuno ha provato?
@Borruso riesci a verificare? Grazie
ciao io ho fatto dei test. Con le fatture di vendita tutto ok, mentre con fatture fornitore estero che generano autofattura non mi fa mai confermare. anche se metto solo in bozza e poi confermo genera un errore Ho provato a eliminare xml originale e confermala per confrontare i due xml a mano e sono identici Qualcuno ha provato?
@Borruso riesci a verificare? Grazie
@eLBati abbiamo visto l'altro giorno assieme @Borruso e @MarcoCalcagni. Mi pare che Giuseppe abbia individuato il motivo per cui si scatena questa anomalia, ma vediamo se ce lo conferma
scusa @eLBati per errore pensavo fosse la tua la review in sospeso, invece è di @marcelofrare
Se @tafaRU non ha nulla in contrario
procedete pure, le mie richieste sono state risolte :+1:
This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-2966-by-eLBati-bump-minor, awaiting test results.
@eLBati your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2966-by-eLBati-bump-minor.
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.