Skip to content

chore: use matomo container#261

Merged
vitormattos merged 1 commit into
mainfrom
chore/use-matomo-container
Jul 29, 2025
Merged

chore: use matomo container#261
vitormattos merged 1 commit into
mainfrom
chore/use-matomo-container

Conversation

@vitormattos

Copy link
Copy Markdown
Member

The previous configuration of Matomo was using the siteId and by this way the setting is totally handmade at website. If we need to add a new setting, will be necessary change the tag.

Using container, the tag configuration is made at Matomo side and not at website side.

The previous configuration of Matomo was using the siteId and by this
way the setting is totally handmade at website. If we need to add a new
setting, will be necessary change the tag.

Using container, the tag configuration is made at Matomo side and not at
website side.

Signed-off-by: Vitor Mattos <vitor@php.rio>
@vitormattos
vitormattos requested a review from Any97Cris June 26, 2025 14:14
@vitormattos vitormattos self-assigned this Jun 26, 2025
@vitormattos
vitormattos marked this pull request as ready for review June 26, 2025 14:14
Comment thread config.php

return [
'production' => false,
'matomo_container' => 'Ux1Y5m98_dev_08f02616a270a3d80b35beb7',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This value, could be in the .env document?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's a good idea.
Thinking more about....
This isn't a secret and stay and is the developer value. If we put at an env file will be a bit more hard to share this value.

@vitormattos
vitormattos merged commit d009407 into main Jul 29, 2025
2 checks passed
@vitormattos
vitormattos deleted the chore/use-matomo-container branch July 29, 2025 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants