-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Renames CustomPiwikJs plugin to CustomJsTracker #15505
Conversation
469afe3
to
21efdb2
Compare
@sgiehl there used to be a plugin on the marketplace named |
Guess it would be overwritten, but not sure if some old files might remain 🤷♂ |
Up to you. If we keep the name, might be worth testing what happens when upgrading from Piwik 2 and such a plugin existed. Might be easier to maybe rename it? |
21efdb2
to
a3282d6
Compare
renamed it again. Need to update the UI tests again, I guess. But should be ready for another review |
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.
Left a few minor comments, nothing major. Mostly around docs.
Adjusted already 6 or so internal plugins. Be good to make the changes in tag manager and resolve the open issues then be good to merge
3eafb10
to
af1f06d
Compare
@tsteur applied the feedback |
Looks good in general @sgiehl some tests will need fixing though |
Guess tests are failing because the changes in tagmanager are needed. will push a submodule change to prove that. |
1c48c6a
to
edfdeda
Compare
Tests seem to pass again. Before merging we need to merge matomo-org/tag-manager#220 and update the submodule ref here. |
f136f64
to
da4a15b
Compare
fixes #13604