Skip to content
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

Improve the way of handling analytics #1784

Closed
curquiza opened this issue Oct 7, 2021 · 3 comments · Fixed by #1800
Closed

Improve the way of handling analytics #1784

curquiza opened this issue Oct 7, 2021 · 3 comments · Fixed by #1800
Assignees
Labels
impacts docs This issue involves changes in the Meilisearch's documentation impacts integrations This issue involves changes in the Meilisearch's integrations tracking issue Tracks development of a global issue
Milestone

Comments

@curquiza
Copy link
Member

curquiza commented Oct 7, 2021

We currently send the analytics directly to Amplitude.
The alternative we want to implement is to push our analytics into Segment instead so that any other analytics tools like Amplitude can be plugged.

@meilisearch/docs-team: thanks to @gmourier, this issue will be updated with the related changes introduced with this PR, for instance, the exhaustive list of the data we send to Segment.

@meilisearch/integration-team: the SDKs will now be able to send information via the User-Agent header.

@curquiza curquiza added tracking issue Tracks development of a global issue impacts docs This issue involves changes in the Meilisearch's documentation impacts integrations This issue involves changes in the Meilisearch's integrations labels Oct 7, 2021
@curquiza curquiza added this to the v0.24.0 milestone Oct 7, 2021
@gmourier
Copy link
Member

gmourier commented Oct 7, 2021

One of the major advantages is that we can control what happens with a vigilance layer to guarantee the privacy of ours users.

That is, if someone makes a mistake one day and sends data deemed personal, we are alerted and we can purge this data very easily. The core is less concerned because we don't have access to any personal data but I think about other teams in the future.

The processes put in place to decide and implement a new metric should prevent this from happening, but prevention is better than cure.

Moreover, if a user decides to delete the data that we could have collected because he didn't deactivate the analytics, he will just have to provide us with his unique identifier generated at the first launch of the binary, so that we dispatch the deletions automatically from Segment and thus being confident that the data have been deleted in addition to ease this operation.

We are also working on a tracking plan that will allow us to instantly visualize everything that is tracked at a given time. We will remove metrics over time that have allowed us to improve the product and that are no longer useful.

@curquiza
Copy link
Member Author

curquiza commented Oct 16, 2021

📣 Here is the current re-work of the spec related to this issue: meilisearch/specifications#84

@curquiza
Copy link
Member Author

@meilisearch/integration-team: the analytics should be disabled from all the integration CIs. This is currently the case except for the devops tools

@bors bors bot closed this as completed in c32f13a Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impacts docs This issue involves changes in the Meilisearch's documentation impacts integrations This issue involves changes in the Meilisearch's integrations tracking issue Tracks development of a global issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants