l10n-italy icon indicating copy to clipboard operation
l10n-italy copied to clipboard

[14.0][IMP] l10n_it_fatturapa_in: fattura semplificata

Open TheMule71 opened this issue 2 years ago • 12 comments

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 avatar Aug 16 '22 11:08 TheMule71

@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.

stevech091 avatar Aug 16 '22 13:08 stevech091

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.

TheMule71 avatar Aug 16 '22 14:08 TheMule71

Anche la preview non funziona, mi esce la pagina bianca.

Preview implementato.

TheMule71 avatar Aug 16 '22 16:08 TheMule71

@stevech091 Aumentata la probabilità di riuscire a creare il partner (se le info ci sono).

TheMule71 avatar Aug 16 '22 16:08 TheMule71

Per la cronaca, i test falliscono, invece caricando a mano lo stesso file usato dal test su runboat, funziona (e crea il contatto).

TheMule71 avatar Aug 16 '22 20:08 TheMule71

@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' "

stevech091 avatar Aug 17 '22 05:08 stevech091

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...

TheMule71 avatar Aug 17 '22 09:08 TheMule71

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?

stevech091 avatar Aug 17 '22 12:08 stevech091

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.

TheMule71 avatar Aug 17 '22 13:08 TheMule71

Oh, dopo lunga lotta con i test, tutto verde. :)

TheMule71 avatar Aug 17 '22 16:08 TheMule71

@TheMule71 ci sono conflitti

eLBati avatar Sep 16 '22 05:09 eLBati

@TheMule71 ci sono conflitti

A posto, grazie.

TheMule71 avatar Sep 16 '22 17:09 TheMule71

Buongiorno a tutti. Cosa manca a questa PR per essere mergiata?

CiroBoxHub avatar Nov 03 '22 13:11 CiroBoxHub

Buongiorno a tutti. Cosa manca a questa PR per essere mergiata?

Tecnicamente, review da parte di mantainer/PSC.

TheMule71 avatar Nov 03 '22 17:11 TheMule71

@sergiocorato se non ricordo male avevi detto che l'avresti testata con qualcuno dei file di un tuo cliente. Hai modo di farlo? Grazie.

tafaRU avatar Jan 17 '23 11:01 tafaRU

@TheMule71 secondo te questa necessita di ulteriori test funzionali?

tafaRU avatar Feb 10 '23 15:02 tafaRU

Mah, considera che io non la uso. Se chi l'ha provata ha visto che funziona, per me si può procedere a review tecnica.

TheMule71 avatar Feb 10 '23 17:02 TheMule71

Conflitti risolti.

TheMule71 avatar May 12 '23 09:05 TheMule71

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). 🤖

OCA-git-bot avatar May 12 '23 10:05 OCA-git-bot

/ocabot merge minor

eLBati avatar May 19 '23 07:05 eLBati

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.

OCA-git-bot avatar May 19 '23 09:05 OCA-git-bot

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.

OCA-git-bot avatar May 19 '23 10:05 OCA-git-bot

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.

OCA-git-bot avatar May 19 '23 10:05 OCA-git-bot

Congratulations, your PR was merged at 616d79543f2a9608887b86b7c5b427d30c245968. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar May 19 '23 11:05 OCA-git-bot