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.

Sign up for GitHub

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

Jump to bottom

Remove sourceMappingURL Comment #55

Closed
TigerC10 opened this issue Sep 11, 2014 · 10 comments
Closed

Remove sourceMappingURL Comment #55

TigerC10 opened this issue Sep 11, 2014 · 10 comments

Comments

@TigerC10
Copy link

I'm compiling all of my JS files into a single file with grunt, and the following line is forcing the browser to attempt to download a file that does not exist:
//# sourceMappingURL=json-patch.min.js.map

Manually specifying this in the file is not required, any browser that supports map files will automatically look for them in the same directory. It's only required if you want to put the map files in a different directory.
warpech added a commit that referenced this issue Nov 26, 2014
@warpech
remove source maps, because they cause more trouble than benefit (#55)…
8a72fcd 
…. So did jQuery: http://blog.jquery.com/2014/01/24/jquery-1-11-and-2-1-released/
@warpech
Copy link
Collaborator

warpech commented Nov 26, 2014

Removed source maps from master branch. Soon to be released as version 0.5.0

@warpech warpech closed this as completed Nov 26, 2014
@warpech
Copy link
Collaborator

warpech commented Nov 28, 2014

Released in 0.5.0

tomalec added a commit that referenced this issue Dec 3, 2014
@tomalec
Remove last missing source map trace #55
8e61317
@adamgen
Copy link

adamgen commented Mar 2, 2015

How and where?

@TigerC10
Copy link
Author

TigerC10 commented Mar 2, 2015

@OLeggen as of version 0.5.0 there's no longer an issue with sourceMappingURL.

@adamgen
Copy link

adamgen commented Mar 2, 2015

I'm on prepros 5.2.0 and it seems to be the issue here:
http://vetbakfar.co.il/wp/

@warpech
Copy link
Collaborator

warpech commented Mar 2, 2015

@OLeggen sorry but this does not have anything to do with the library hosted in this repository. Perhaps you are searching for Prepros discussion forums: http://discuss.prepros.io/

@adamgen
Copy link

adamgen commented Mar 2, 2015

Ho. right -thanks :)

@xiaomao155
Copy link

我也是这种问题 谁能帮帮我

@TigerC10
Copy link
Author

这是解决

@gdibble
Copy link

gdibble commented Apr 17, 2015

🎉

@tomalec tomalec mentioned this issue Sep 8, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@TigerC10 @warpech @gdibble @xiaomao155 @adamgen