[18.0][MIG] website_analytics_matomo: Migration to 18.0#1107
[18.0][MIG] website_analytics_matomo: Migration to 18.0#1107ByteMeAsap wants to merge 18 commits intoOCA:18.0from
Conversation
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-15.0/website-15.0-website_analytics_matomo Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_analytics_matomo/
Currently translated at 50.0% (6 of 12 strings) Translation: website-15.0/website-15.0-website_analytics_matomo Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_analytics_matomo/it/
Currently translated at 63.1% (12 of 19 strings) Translation: website-15.0/website-15.0-website_analytics_matomo Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_analytics_matomo/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-16.0/website-16.0-website_analytics_matomo Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_analytics_matomo/
Currently translated at 100.0% (19 of 19 strings) Translation: website-16.0/website-16.0-website_analytics_matomo Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_analytics_matomo/it/
ByteMeAsap
force-pushed
the
18.0-mig-website_analytics_matomo
branch
2 times, most recently
from
91afb33 to
78e2163
Compare
View Fixes Increase Test Coverage
ByteMeAsap
force-pushed
the
18.0-mig-website_analytics_matomo
branch
from
78e2163 to
4f087b1
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
|
hi @ByteMeAsap. could you reopen ? I use this module in production, and it works like a charm. @victor-champonnois : could you review this PR, as you are the original author ? thanks ! |
|
hey @legalsylvain , can you please guide me on how to reopen this? |
|
@OCA/website-maintainers could you reopen ? |
|
@legalsylvain if this works like a charm for you, you should review it #1133 seems interesting to review+integrate too |
It is not possible, when it's closed. but now, it's possible. |
legalsylvain
left a comment
legalsylvain
left a comment
There was a problem hiding this comment.
LGTM. Used in production. Thanks for porting this tool.
One thing :
-
this module implement the "integrate js file" way. As a result, we have full information in matomo, but such implementation is blocked by tools that preserves user privacy (Like uBlock Origin). As a result, all people using such tool will be ignored, in matomo, and user count will be wrong. That's a pity.
-
Another way to implement is to use the "matomo API" way. (here Odoo calls directly matomo api). As a result, user count is correct. (but there will be less information in matomo like no duration of user in each page, etc...)
What do you think ?
Could you just add a ROADMAP.md file to mention the interest to implement the second way ?
Thanks !
github-actions
bot
removed
the
stale
11 participants
No description provided.