dojo 1.7.0 #95

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@william-gr

No description provided.

@klipstein
Owner

Did you test your environment with Dojo 1.7.0? With this patch Dojango fails when loading the dojo.xd.js from google. Dojo 1.7.0 completely got rid of the separation of the xd/non-xd loader conecpt and the following URL is generated by dojango:

http://ajax.googleapis.com/ajax/libs/dojo/1.7.0/dojo/dojo.xd.js

The right URL would be:

http://ajax.googleapis.com/ajax/libs/dojo/1.7.0/dojo/dojo.js

And it seems that dojango needs a bigger patch to reflect the complete new loading mechanism of dojo.

@william-gr
@klipstein
Owner

I've fixed Dojango to use Dojo 1.7.0 here:

a5af583...master

Thanks for remembering me to upgrade to Dojo 1.7.0. I'm not applying your patch because it was non-functional (as commented before: the simple version setting of 1.7.0 will generate the non-existent URL http://ajax.googleapis.com/ajax/libs/dojo/1.7.0/dojo/dojo.xd.js). Closing the issue, because Dojango is able to handle Dojo 1.7.0 now.

@klipstein klipstein closed this Dec 7, 2011
@william-gr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment