search

OCA / intrastat-extrastat

GNU Affero General Public License v3.0
35 stars 107 forks source link

[16.0][FIX] intrastat_product: Display note field as html on product declat… #248

Closed aritzolea closed 10 months ago

aritzolea commented 10 months ago

…ation

Note field on intrastat.product.declaration is currently being displayed as raw html:

Screenshot from 2023-11-22 15-24-58

Adds html widget so it is displayed correctly:

Screenshot from 2023-11-22 15-26-32

pedrobaeza commented 10 months ago

Maybe not all localizations are using HTML content. Can you confirm, @jdidderen-noviat ?

jdidderen-noviat commented 10 months ago

It should be good on the Belgian side, I already adapted to the changes made by Alexis during the migration to 16.0. In my opinion, not an issue to add the html widget

aritzolea commented 10 months ago

@pedrobaeza Bot didn't apply merge

pedrobaeza commented 10 months ago

/ocabot merge minor

OCA-git-bot commented 10 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-248-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 10 months ago

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

alexis-via commented 8 months ago

I think we should have moved note from fields.Text() to fields.Html() instead of just changing the widget...