8000 Update Matomo tracking script by stevepiercy · Pull Request #944 · plone/training · GitHub
[go: up one dir, main page]

Skip to content

Update Matomo tracking script - #944

Merged
fredvd merged 3 commits into
2024from
2024-deploy-fixes
Sep 19, 2026
Merged

fredvd merged 3 commits into
2024from
2024-deploy-fixes

Conversation

@stevepiercy
@stevepiercy stevepiercy commented Apr 7, 2025 •
Copy link
Copy Markdown
Member

📚 Documentation preview 📚: https://plone-training--944.org.readthedocs.build/

@fredvd
fredvd commented Apr 7, 2025
Copy link
Copy Markdown
Member

Note to self @fredvd : need to check why tag container is used on 2023 and repeat for 2024 setup if required

@stevepiercy
Copy link
Copy Markdown
Member Author

@fredvd reping. Conference trainings in one month, and this is now time-sensitive.

@stevepiercy
Copy link
Copy Markdown
Member Author

@fredvd would you please review this PR? Do I need to revise it? Thank you!

@fredvd
fredvd commented Jul 31, 2026
Copy link
Copy Markdown
Member

Hi Steve,

Sorry, totally missed this one. Let me check this one. We switch from a mtm (matomo tag manager inclusion) to a direct matomo tag. I can't remember the specifics anymore, did we decide not to use the tag manager container anymore to update the tag?

@stevepiercy
Copy link
Copy Markdown
Member Author

@fredvd if I was part of that decision, I don't recall. The best I can suggest is to do a search in the Plone organization, and maybe that rings a bell.

https://github.com/search?q=org%3Aplone+matomo&type=issues

@fredvd
fredvd commented Sep 19, 2026
Copy link
Copy Markdown
Member

Found the cause of the confusion here. We don't remember, because our matomo is configured to create a new container automatically when a site is created. So we have been using mtm for some years now for training sites, using the default container generated, but you have to go to tag manager first and pick up the tag container code from there.

@fredvd fredvd left a comment
8000
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated to use auto generated tag container for the 2024.training.plone.org

@fredvd
fredvd merged commit c99b42a into 2024 Sep 19, 2026
3 checks passed
@fredvd
fredvd deleted the 2024-deploy-fixes branch September 19, 2026 16:01
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Plone Documentation Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants

0