Skip to content

Fix: CSP compliance in Backbone and DOMElementModifications #687

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix: CSP compliance in Backbone and DOMElementModifications #687

wants to merge 1 commit into from

Conversation

oliverfoster
Copy link
Member

fixes #686

Partial fix.

Updated

  • Removed IE11 and browsers without Class support
  • Changed all eval and Function use to alternatives

@oliverfoster oliverfoster self-assigned this Aug 22, 2025
cahirodoherty-learningpool
cahirodoherty-learningpool approved these changes Aug 22, 2025
View reviewed changes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

@joe-allen-89 joe-allen-89 moved this from New to Needs Reviewing in adapt_framework: The TODO Board Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chris-steele chris-steele chris-steele approved these changes

@cahirodoherty-learningpool cahirodoherty-learningpool cahirodoherty-learningpool approved these changes

Assignees

@oliverfoster oliverfoster

Labels

bug

Projects

adapt_framework: The TODO Board
Status: Needs Reviewing

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CSP compliance

3 participants

@oliverfoster @chris-steele @cahirodoherty-learningpool