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

Document few changes for Matomo 4 #353

Merged
merged 2 commits into from
Jun 18, 2020
Merged

Document few changes for Matomo 4 #353

merged 2 commits into from
Jun 18, 2020

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Jun 18, 2020

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.

@tsteur
Copy link
Member Author

tsteur commented Jun 18, 2020

@sgiehl or @Findus23 if someone could have a very quick look that be great.

@tsteur tsteur added this to the Current sprint milestone Jun 18, 2020
Copy link
Member

@sgiehl sgiehl left a 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),
Copy link
Member

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?

Copy link
Member Author

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

Copy link
Member

@Findus23 Findus23 left a 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.

@tsteur tsteur merged commit b0fe1e1 into master Jun 18, 2020
@tsteur tsteur deleted the matomo4docs branch June 18, 2020 20:32
@innocraft-automation innocraft-automation removed this from the Current sprint milestone Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants