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.

Sign up for GitHub

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

Jump to bottom

DOMPurify is not defined #94

Open
2 tasks done
alagane opened this issue Nov 9, 2021 · 0 comments
Open
2 tasks done

DOMPurify is not defined #94

alagane opened this issue Nov 9, 2021 · 0 comments
Milestone
Polish sprint

Comments

@alagane
Copy link
Member

alagane commented Nov 9, 2021 •
edited
Loading

When loading the application, DOMPurify is not defined.

This is caused by a missing require on esn-frontend-common-libs.
This bug is present because it was tested on esn-frontend-inbox where DOMPurify is added by webpack, thus the require is not needed here.

We have to:
@alagane alagane added this to the Polish sprint milestone Nov 9, 2021
alagane added a commit to alagane/esn-frontend-common-libs that referenced this issue Nov 9, 2021
@alagane
Add missing require for DOMPurify
586dc68 
Related to linagora/esn-frontend-mailto#94
@alagane alagane mentioned this issue Nov 9, 2021
Merged
alagane added a commit to alagane/esn-frontend-common-libs that referenced this issue Nov 9, 2021
@alagane
Add missing require for DOMPurify
4742dde 
Related to linagora/esn-frontend-mailto#94
alagane added a commit to alagane/esn-frontend-common-libs that referenced this issue Nov 9, 2021
@alagane
Add missing require for DOMPurify
7f2e1ed 
Related to linagora/esn-frontend-mailto#94
alagane added a commit to alagane/esn-frontend-common-libs that referenced this issue Nov 9, 2021
@alagane
Add missing require for DOMPurify
3824f63 
Related to linagora/esn-frontend-mailto#94
alagane added a commit to alagane/esn-frontend-inbox that referenced this issue Nov 15, 2021
@alagane
Remove DOMPurify dependency
ed90e3c 
Related to linagora/esn-frontend-mailto#94
@alagane alagane mentioned this issue Nov 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Polish sprint
Development

No branches or pull requests
1 participant
@alagane