[14.0][FIX] assets_management view html report

[sergiocorato]

No description provided.

[TheMule71]

Da codice capisco che hai integrato la generazione dei preview html con gli altri due report pdf e xlsx, di fatto togliendo un caso particolare che ritornava una action.
La domanda è:

• fixa qualcosa?
• aggiunge qualcosa?
• è un refactoring puramente estetico? (tradotto: non cambia nulla per l'utente, è solo il codice ad essere più pulito)

[sergiocorato]

Da codice capisco che hai integrato la generazione dei preview html con gli altri due report pdf e xlsx, di fatto togliendo un caso particolare che ritornava una action. La domanda è:

* fixa qualcosa?

* aggiunge qualcosa?

* è un refactoring puramente estetico? (tradotto: non cambia nulla per l'utente, è solo il codice ad essere più pulito)

Il bottone non funzionava proprio (creo una issue in cui riporto le varie PR sugli asset).

[sergiocorato]

Da codice capisco che hai integrato la generazione dei preview html con gli altri due report pdf e xlsx, di fatto togliendo un caso particolare che ritornava una action. La domanda è:

* fixa qualcosa?

* aggiunge qualcosa?

* è un refactoring puramente estetico? (tradotto: non cambia nulla per l'utente, è solo il codice ad essere più pulito)

Il bottone non funzionava proprio (creo una issue in cui riporto le varie PR sugli asset).

#3351

[Marianna-Marasco]

Tasto "vedi" ora funziona! ;-)

[matteoopenf]

provato su runbot, per me si puo' mergiare

[eLBati]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3348-by-eLBati-bump-patch, awaiting test results.

[OCA-git-bot]

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