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.

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

JS Tracker: remove JSON2 #15413

Merged
merged 3 commits into from Jan 20, 2020
Merged

JS Tracker: remove JSON2 #15413

merged 3 commits into from Jan 20, 2020

Conversation

Findus23
Copy link
Member

this is the same PR as #11031 and finally fixes #6093

➜  ~/public_html/piwik git:(4.x-dev) ✗ l piwik.*
Permissions Size User  Date Modified Name
.rw-r--r--   68k lukas 17 Jän 16:47  piwik.js
.rw-r--r--   59k lukas 17 Jän 16:47  piwik.old.js
➜  ~/public_html/piwik git:(4.x-dev) ✗ gzip piwik.*
➜  ~/public_html/piwik git:(4.x-dev) ✗ l piwik.*
Permissions Size User  Date Modified Name
.rw-r--r--   23k lukas 17 Jän 16:47  piwik.js.gz
.rw-r--r--   19k lukas 17 Jän 16:47  piwik.old.js.gz

@Findus23 Findus23 added the Needs Review PRs that need a code review label Jan 17, 2020
@Findus23 Findus23 added this to the 4.0.0 milestone Jan 17, 2020
@tsteur tsteur merged commit 9e24966 into 4.x-dev Jan 20, 2020
@tsteur tsteur deleted the remove-json2 branch January 20, 2020 21:12
@Findus23 Findus23 removed the Needs Review PRs that need a code review label Jan 20, 2020
@sgiehl sgiehl mentioned this pull request Jan 25, 2020
Closed
@tsteur tsteur mentioned this pull request Jan 26, 2020
Merged
@mattab mattab changed the title remove JSON2 JS Tracker: remove JSON2 Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.0.0
Development

Successfully merging this pull request may close these issues.

Clean up JSON2 in piwik.js
2 participants
@Findus23 @tsteur