-
-
Notifications
You must be signed in to change notification settings - Fork 339
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 analytics wiring #4049
Add analytics wiring #4049
Conversation
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.
LGTM! I'm curious about why the MATHESAR_ANALYTICS_URL
isn't added in the docker-compose.yml
Co-authored-by: Anish Umale <[email protected]>
I plan to remove that environment variable anyway. We'll hardcode the URL in the settings file. But, I don't want to publish the URL until we have some DDOS, etc. protection on it. So, I set it up so a core team developer can just use an environment variable while testing without risking accidentally committing the URL in the settings file. |
Related to #4063
This adds some machinery to collect and transmit analytics data reports to a URL specified in the settings.
Technical details
AnalyticsReport
model shows all information collected.AnalyticsReport
model once per day.Checklist
Update index.md
).develop
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin