-
Notifications
You must be signed in to change notification settings - Fork 62
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 script and connect it with base template #320
Conversation
Thank you for this work. I tested it locally but I assume that we need to connect to matomo.qgis.org to see if it works. Besides, I have a few suggestions about the integration of matomo in QGIS-Django:
We could also ask @dimasciput about how the environment variables are implemented in production. |
@Xpirix, thanks for the tip to use the plugin, seems like a good idea, I'll try to use it |
@Xpirix, I installed motomo-plugin according to its documentation, although I had no luck getting the project to work after installing it. Maybe I'm doing something wrong. Could you please take a look? Below is the error I'm getting Log
I have a suspicion that django 2 is too old for this plugin. |
Hello @Akiyamka , I've checked it locally and had the same error too. Adding |
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #320 +/- ##
==========================================
+ Coverage 81.94% 81.96% +0.01%
==========================================
Files 139 139
Lines 5418 5422 +4
==========================================
+ Hits 4440 4444 +4
Misses 978 978 ☔ View full report in Codecov by Sentry. |
Looks good to me. Thank you @Akiyamka. |
@dimasciput can you review, merge deploy please? |
@dimasciput can you link to the policy in question? |
No description provided.