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

Manipulation: support data-URI scripts insertion #1888

Closed
wants to merge 1 commit into from

Conversation

rhyzx
Copy link
Contributor

@rhyzx rhyzx commented Nov 28, 2014

Fixes gh-1887

@markelog
Copy link
Member

Add spaces around quotes in

jQuery("#qunit-fixture")

and

window["testFoo"]

@markelog
Copy link
Member

In CLA, If your real name not "rhyzx" you have to write a correct one.

@rhyzx rhyzx force-pushed the fix-data-uri-script-insertion branch from 5fd5e6e to d091195 Compare November 29, 2014 11:21
@rhyzx
Copy link
Contributor Author

rhyzx commented Nov 29, 2014

updated, and i've signed up a new CLA with my realname,
thanks @markelog

@markelog
Copy link
Member

markelog commented Dec 3, 2014

Sorry about this bureaucratic stuff, but name in CLA should match the commit author too.

Could you change that? After that we're good to go

@markelog
Copy link
Member

markelog commented Dec 3, 2014

With another thought: i could that for you, if you're okay with that

@rhyzx rhyzx force-pushed the fix-data-uri-script-insertion branch from d091195 to bdd5786 Compare December 3, 2014 02:15
@rhyzx
Copy link
Contributor Author

rhyzx commented Dec 3, 2014

🐱 DONE

@markelog markelog closed this in 15f4dec Dec 3, 2014
@markelog
Copy link
Member

markelog commented Dec 3, 2014

Merged

@markelog markelog mentioned this pull request Nov 16, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

data-URI scripts insertion
2 participants