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

Invert escapeHtml option -> dangerouslyAllowHtml #484

Closed
wooorm opened this issue Oct 16, 2020 · 0 comments
Closed

Invert escapeHtml option -> dangerouslyAllowHtml #484

wooorm opened this issue Oct 16, 2020 · 0 comments
Labels
💪 phase/solved Post is done 🦋 type/enhancement This is great to have

Comments

@wooorm
Copy link
Member

wooorm commented Oct 16, 2020

Subject of the feature

This option can now be set to false to live dangerously.
That’s always a bit confusing to me. Using an option that has dangerous in the name, and uses true to be dangerous, is clearer to me.
Can be a smooth update with a default to the current escapeHtml

@wooorm wooorm added 🙉 open/needs-info This needs some more info 🦋 type/enhancement This is great to have labels Oct 16, 2020
@wooorm wooorm closed this as completed in 7a5e3a1 Oct 17, 2020
@wooorm wooorm added ⛵️ status/released and removed 🙉 open/needs-info This needs some more info labels Oct 19, 2020
@wooorm wooorm added the 💪 phase/solved Post is done label Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 phase/solved Post is done 🦋 type/enhancement This is great to have
Development

No branches or pull requests

1 participant