Added public urlArgs method (as suggested in #66)

[boye]

urlArgs makes it possible to concat a querystring to all scripts, e.g. for cache busting purposes.

[ded]

@boye cool!. let me review it and i'll send some thoughts shortly

[houston88]

Hey this is great! Beats having to come up with a complicated grunt build to hash our file names.

[ded]

ah! cool. with a PR and everything. i'll have a look

[houston88]

Wanted to mention I get the following error sometimes on my main index.html when I run with the urlArgs changes in the pull request... I could be doing something wrong. The url args are getting appended to the request for the main page for some reason. The page still loads fine.

Resource interpreted as Script but transferred with MIME type text/html: "https://local.blah.comp.com:8443/login/?v=1403280636567". script.js:84
create script.js:84
loading script.js:67
(anonymous function) script.js:32
every script.js:27
each script.js:31
(anonymous function) script.js:55

Uncaught SyntaxError: Unexpected token < local.blah.comp.com/:1

[boye]

I am using the urlArgs functionality in a production environment for several weeks now...without any problems :)

[houston88]

I think it has something to do with angular, happens after angular lib is loaded. In any case, adding the following to the path conditional fixed for me. if (path === null || path === '') return callback()

Anyone see any gotchas with that change?

[ded]

that would be a solid change. it's always another lib do something weird messing it up for others

[houston88]

Doh my bad, turned out to be something we were doing. No worries with the
existing code.

Thanks, -Houston
On Jun 25, 2014 7:15 PM, "Dustin Diaz" notifications@github.com wrote:

that would be a solid change. it's always another lib do something weird
messing it up for others

—
Reply to this email directly or view it on GitHub
#67 (comment).

[boye]

@houston88 good to know, thanks for testing it! Now it's up to @ded to merge the pull request ;)