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

Don’t apply the CDN prefix on cordova. For #3278 #3311

Merged
merged 1 commit into from Dec 18, 2014

Conversation

Projects
None yet
2 participants
@tmeasday
Contributor

tmeasday commented Dec 18, 2014

Cordova serves assets directly from the device so it doesn’t make sense to hit the CDN for them. Beside it causes issues because they are served from /__cordova anyway.

@tmeasday tmeasday changed the title from Don’t apply the CDN prefix on cordova. #3278 to Don’t apply the CDN prefix on cordova. For #3278 Dec 18, 2014

Don’t apply the CDN prefix on cordova. #3278
Cordova serves assets directly from the device so it doesn’t make sense to hit the CDN for them. Besides it causes issues because they are served from /__cordova (on the real webserver) anyway.

@tmeasday tmeasday force-pushed the tmeasday:cordova-cdn-fix branch from 69a14a0 to b045093 Dec 18, 2014

Slava added a commit that referenced this pull request Dec 18, 2014

Merge pull request #3311 from tmeasday/cordova-cdn-fix
Don’t apply the CDN prefix on cordova. For #3278

@Slava Slava merged commit 0d4c587 into meteor:devel Dec 18, 2014

@Slava

This comment has been minimized.

Member

Slava commented Dec 18, 2014

thanks, merged

@tmeasday tmeasday deleted the tmeasday:cordova-cdn-fix branch Dec 22, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment