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
Add Matomo Tag Manager to core #13402
Conversation
There's a failing system test in Anonymous tracking which will be fixed when merging (to avoid merging PR in submodules etc). There are couple of failing UI tests but I believe some of them fail in 3.x-dev as well. I've been trying to reproduce and fix this failing screenshot https://builds-artifacts.matomo.org/matomo-org/matomo/tagmanager/29950/UIIntegrationTest_admin_manage_tracking_code.png but cannot reproduce it locally when running the UI tests and neither in the browser. Also not seeing any JS error in the UI build test logs. |
FYI: I'm currently waiting for a review of the wording on that page. |
Updated wording and added translations. |
screenshots need to be updated, otherwise lgtm (tested locally and it works) |
refs matomo-org/matomo#13402 Haven't tested it but I reckon it should work
No description provided.