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
[13.0][MIG] website_analytics_piwik #678
[13.0][MIG] website_analytics_piwik #678
Conversation
Currently translated at 77.8% (7 of 9 strings) Translation: website-11.0/website-11.0-website_analytics_piwik Translate-URL: https://translation.odoo-community.org/projects/website-11-0/website-11-0-website_analytics_piwik/pt_BR/
Currently translated at 66.7% (6 of 9 strings) Translation: website-11.0/website-11.0-website_analytics_piwik Translate-URL: https://translation.odoo-community.org/projects/website-11-0/website-11-0-website_analytics_piwik/es/
a71f5fe
to
3172335
Compare
What about renaming the module to |
@OCA/website-maintainers Piwik was renamed to Matomo for a couple of years now. Should I change the technical name of the module? |
website_analytics_piwik/migrations/11.0.1.0.0/post-migration.py
Outdated
Show resolved
Hide resolved
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.
I agree renaming would be nice, but seeing how old this PR is I won't bother you with it. Please just put a note on the roadmap or somewhere that we should actually do it for v14 when you follow up on the other remarks
3172335
to
4cadf16
Compare
[FIX] Black [ADD] Rename to Matomo note [FIX] Use bootstrap 4 class [REM] 11.0 migration script [FIX] Prettify xml
4cadf16
to
a8436fb
Compare
Can this be merged, please. |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at da397a8. Thanks a lot for contributing to OCA. ❤️ |
No description provided.