-
Notifications
You must be signed in to change notification settings - Fork 87
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
Document few changes for Matomo 4 #353
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -107,7 +107,7 @@ _paq.push(['AbTesting::create', { | |||
} | |||
return false; | |||
}, | |||
matomoTracker: Matomo.getAsyncTracker(piwikUrl, piwikSiteId), | |||
piwikTracker: Matomo.getAsyncTracker(matomoUrl, matomoSiteId), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guess the piwikTracker
option hasn't been renamed in the plugin yet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exactly, will change the premium feature docs once we renamed things there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't see anything weird when scrolling through. Maybe once we have replaced all piwik occurrences, we could remove the automatic replacement as I think it still breaks some pages.
refs matomo-org/matomo#8415
refs matomo-org/matomo#12420
Updated some docs based on the changelog and also updated some domains, piwik.js and matomo.js mentions etc.