Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Catalog Item Long Description is not parsing html #23224

Closed
eggoide opened this issue Oct 3, 2024 · 5 comments
Closed

Catalog Item Long Description is not parsing html #23224

eggoide opened this issue Oct 3, 2024 · 5 comments
Assignees
Labels

Comments

@eggoide
Copy link

eggoide commented Oct 3, 2024

Hi all,

in petrosian-1.20230912222549_a07cb46 , the long descriptions are no longer parsed as html but just as plain text.
It is not possible to format the text in description in any way. Is there any fix or workaround? It used to work in previous
versions of ManageIQ.

long1 long2

Thank you

@eggoide eggoide added the bug label Oct 3, 2024
@Fryguy
Copy link
Member

Fryguy commented Oct 3, 2024

Hi @eggoide ... This was an intentional decision to avoid various security issues that can happen with supporting HTML. Instead, we support Markdown in that Long Description, which should give you just about anything you could need, so in your case use **test** (test) as the text.

@Fryguy Fryguy closed this as completed Oct 3, 2024
@eggoide
Copy link
Author

eggoide commented Oct 3, 2024

@Fryguy Thank you for the answer, is there something that needs to be put in the description editor to tell it it is in markdown?
Because it is not parsing markdown either:

long3 long4

@Fryguy
Copy link
Member

Fryguy commented Oct 3, 2024

That's strange - which version are you using?

@Fryguy
Copy link
Member

Fryguy commented Oct 3, 2024

@eggoide It looks like this landed in quinteros: ManageIQ/manageiq-ui-classic#8972

I think the way this came about is we noticed the HTML wasn't working, and then when we went to fix it, it opened up a can of security worms, so we switched to markdown.

@Fryguy Fryguy self-assigned this Oct 3, 2024
@eggoide
Copy link
Author

eggoide commented Oct 3, 2024

@Fryguy We have petrosian-1.20230912222549_a07cb46, so I guess we would have to upgrade to make it work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants