Query String for Jsdelivr | WordPress.org

Resolved grantdb
(@sixer)

9 years ago

Thanks again for a great plugin! I know its not the “release” version of WP but pretty dang close…Anyways I am getting an odd version string added to the cdn url

https://cdn.jsdelivr.net/mootools/1.5.0/mootools-core-nocompat.min.js?ver=4.7-RC2-39491

I think it should be something like,

https://cdn.jsdelivr.net/mootools/1.5.0/mootools-core-nocompat.min.js?ver=1.5.0″

I am not sure if this matters much but possibly would make the browser re-download the mootools even if already in cache from another site?

What do you think?

Thanks!

Viewing 3 replies - 1 through 3 (of 3 total)

Plugin Author Andi Dittrich
(@andi-dittrich)

9 years ago

Of course, you’re right – adding the WPVersion Tag is the default behaviour of WordPress enqueue_scripts hook.

i will explicit remove the version tag in the next release to avoid this (minimal) issue

Thread Starter grantdb
(@sixer)

9 years ago

Great thanks! I didn’t think it was an issue.

Have a great one!

Plugin Author Andi Dittrich
(@andi-dittrich)

9 years ago

changed in current master branch https://github.com/AndiDittrich/WordPress.Enlighter

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Query String for Jsdelivr’ is closed to new replies.

3 replies
2 participants
Last reply from: Andi Dittrich
Last activity: 9 years ago
Status: resolved