@spacenate opened this issue on December 18th 2016

Refs #6093, removes the JSON3 library from piwik.js. More specifically, I removed the global JSON_PIWIK, and replaced all references to it with windowAlias.JSON. I also updated the Incompatibility table comment at the top of the file. I then minified this file as described in js/README.md.

@tsteur commented on December 18th 2016

We are still supporting IE7 and IE6 when it comes to tracking as well as some other old browsers. We could remove JSON3, however we would need to announce it in advance and give people time to work around. For example we would need to give people the possibility to include JSON3 or JSON2 manually and Piwik would need to pick it up if native JSON is not present.

@spacenate commented on December 18th 2016

Ah okay - I was confused by the 3.0.0 milestone on #6093.

We could remove JSON3,

What would you like me to do with this PR?

@mattab commented on December 25th 2016

Ah okay - I was confused by the 3.0.0 milestone on #6093.

oops, we need to take another look at all issues in this milestone. we hope you'll pick another issue to work on :+1: thanks

This issue was closed on December 25th 2016
Powered by GitHub Issue Mirror