-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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(core, web): fix compatibility with TrustedTypes #12383
Conversation
<!-- | ||
If you wanna test the require-trusted-types-for CSP, uncomment the line below | ||
and run the app in a browser that supports Trusted Types (e.g. Chrome 83+). | ||
Note that you need to run the app in release mode till this issue is fixed: | ||
https://github.com/requirejs/requirejs/issues/1832 |
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.
<!-- | |
If you wanna test the require-trusted-types-for CSP, uncomment the line below | |
and run the app in a browser that supports Trusted Types (e.g. Chrome 83+). | |
Note that you need to run the app in release mode till this issue is fixed: | |
https://github.com/requirejs/requirejs/issues/1832 | |
<!-- | |
If you want to test the require-trusted-types-for CSP, uncomment the line below | |
and run the app in a browser that supports Trusted Types (e.g. Chrome 83+). | |
Note that you need to run the app in release mode until this issue is fixed: | |
https://github.com/requirejs/requirejs/issues/1832 |
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.
Made a suggestion to docs, other than that, LGTM
<!-- | ||
If you wanna test the require-trusted-types-for CSP, uncomment the line below | ||
and run the app in a browser that supports Trusted Types (e.g. Chrome 83+). | ||
Note that you need to run the app in release mode till this issue is fixed: |
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.
Note that you need to run the app in release mode till this issue is fixed: | |
Note that you need to run the app in release mode until this issue is fixed: |
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.
Applied
@@ -19,10 +19,18 @@ | |||
<meta charset="UTF-8" /> | |||
<meta content="IE=Edge" http-equiv="X-UA-Compatible" /> | |||
|
|||
<meta | |||
<!-- | |||
If you wanna test the require-trusted-types-for CSP, uncomment the line below |
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.
If you wanna test the require-trusted-types-for CSP, uncomment the line below | |
If you want to test the require-trusted-types-for CSP, uncomment the line below |
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.
Fixed
A new version of |
Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
closes #12301
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
).This will ensure a smooth and quick review process. Updating the
pubspec.yaml
and changelogs is not required.///
).melos run analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?