-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix missing applying of the translate
service
#77
fix missing applying of the translate
service
#77
Conversation
translate
servicetranslate
service
Hi @Vovamzur, thanks for your contribution. Unfortunately, until you sign the CLA, we cannot merge it. |
done |
return description && ( | ||
<Markup | ||
markup={ sanitizeHTML(description) } | ||
markup={ sanitizeHTML(translate(description)) } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly in the required order. We want to sanitize the translation result :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great contribution, thank you so much for implementing this!
I just noticed that the commit messages did not adhere to the conventional commits guidelines which we follow at bpmn.io. I fixed the commits via 911ab0b |
Closes #51