l10n-italy
l10n-italy copied to clipboard
[14.0][IMP] l10n_it_fatturapa_in: fattura semplificata
Implementa #2901 (importazione di fattura semplificata)
Incorpora e dipende da: #2820 - mergiata.
Da fare rebase dopo il merge di #2820. - rebase fatto.
Chi volesse fare review, i primi 8 commit sono quelli di #2820 (per cui meglio fare review di là) - sono stati eliminati.
@TheMule71 ho provato a fare un test di importazione della TD07, ha parzialmente funzionato perchè mi ha creato la fattura MA non il supplier. E' voluto? Anche la preview non funziona, mi esce la pagina bianca.
Beh, per il partner devo vedere, è possibile che manchino informazioni per crearlo. Magari aggiungo un controllo sul partner nel test.
Per il preview, no, non l'ho inserito in effetti, devo caricare il foglio di stile immagino.
Anche la preview non funziona, mi esce la pagina bianca.
Preview implementato.
@stevech091 Aumentata la probabilità di riuscire a creare il partner (se le info ci sono).
Per la cronaca, i test falliscono, invece caricando a mano lo stesso file usato dal test su runboat, funziona (e crea il contatto).
@TheMule71 grazie. Ho fatto un test funzionale e mi sembra un buon risultato; tuttavia volevo cambiare il tipo di preview ma quando sono andato in Configuration mi ha dato questo messaggio di errore. " File "/mnt/data/odoo-addons-dir/l10n_it_fatturapa_out/models/company.py", line 65, in onchange_company_id res = super(AccountConfigSettings, self).onchange_company_id() File "/mnt/data/odoo-addons-dir/l10n_it_fatturapa/models/company.py", line 160, in onchange_company_id company.fatturapa_simple_preview_style or False Exception
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "/opt/odoo/odoo/http.py", line 639, in _handle_exception return super(JsonRequest, self)._handle_exception(exception) File "/opt/odoo/odoo/http.py", line 315, in _handle_exception raise exception.with_traceback(None) from new_cause AttributeError: 'res.config.settings' object has no attribute 'fatturapa_simple_preview_style' "
Non puoi cambiare il tipo di preview, non ho trovato quello assosoftware per la fattura semplificata, per cui esiste solo quello della AdE. Non ho proprio provato il menu di configurazione, ho copiato l'esistente e basta, adesso ci guardo... l'opzione l'ho messa per pura simmetria magari non si sa mai un domani qualcuno farà uno stile alternativo...
Il menu configurazione lo trovi già nel menu della contabilità. Per me puo' anche andare bene la preview che hai messo tu, tuttavia mi viene il dubbio che se hai impostato la preview di assosoftware che succede? Fa lo switch in automatico sullo standard per la fattura semplificata poi ritorna a quello di asso?
Il menu configurazione lo trovi già nel menu della contabilità. Per me puo' anche andare bene la preview che hai messo tu, tuttavia mi viene il dubbio che se hai impostato la preview di assosoftware che succede? Fa lo switch in automatico sullo standard per la fattura semplificata poi ritorna a quello di asso?
Mi sono spiegato male. Avevo (quella era l'idea) aggiunto un'opzione specifica per la f.e. semplificata, in modo da poter decidere di vedere quella ordinaria col preview Assosoftware e quella semplificata con quello AdE. Semplicemente, mancava un pezzo. Adesso dovrebbe essere a posto.
Oh, dopo lunga lotta con i test, tutto verde. :)
@TheMule71 ci sono conflitti
@TheMule71 ci sono conflitti
A posto, grazie.
Buongiorno a tutti. Cosa manca a questa PR per essere mergiata?
Buongiorno a tutti. Cosa manca a questa PR per essere mergiata?
Tecnicamente, review da parte di mantainer/PSC.
@sergiocorato se non ricordo male avevi detto che l'avresti testata con qualcuno dei file di un tuo cliente. Hai modo di farlo? Grazie.
@TheMule71 secondo te questa necessita di ulteriori test funzionali?
Mah, considera che io non la uso. Se chi l'ha provata ha visto che funziona, per me si può procedere a review tecnica.
Conflitti risolti.
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). 🤖
/ocabot merge minor
What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-2907-by-eLBati-bump-minor, awaiting test results.
It looks like something changed on 14.0
in the meantime.
Let me try again (no action is required from you).
Prepared branch 14.0-ocabot-merge-pr-2907-by-eLBati-bump-minor, awaiting test results.
It looks like something changed on 14.0
in the meantime.
Let me try again (no action is required from you).
Prepared branch 14.0-ocabot-merge-pr-2907-by-eLBati-bump-minor, awaiting test results.
Congratulations, your PR was merged at 616d79543f2a9608887b86b7c5b427d30c245968. Thanks a lot for contributing to OCA. ❤️