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

Add domain telemetry on app mount #5631

Merged
merged 8 commits into from
Dec 9, 2020
Merged

Add domain telemetry on app mount #5631

merged 8 commits into from
Dec 9, 2020

Conversation

djhi
Copy link
Contributor

@djhi djhi commented Dec 7, 2020

In production, react-admin applications will send an anonymous request on mount to a telemetry server operated by marmelab. You can see this request by looking at the Network tab of your browser DevTools:

https://react-admin-telemetry.marmelab.com/react-admin-telemetry

The only data sent to the telemetry server is the admin domain (e.g. "example.com") - no personal data is ever sent, and no cookie is included in the response. The react-admin team uses these domains to track the usage of the framework.

You can opt out of telemetry by simply adding disableTelemetry to the <Admin> component:

// in src/App.js
import * as React from "react";
import { Admin } from 'react-admin';

const App = () => (
    <Admin disableTelemetry>
        // ...
    </Admin>
);

@djhi djhi added RFR Ready For Review WIP Work In Progress and removed RFR Ready For Review labels Dec 7, 2020
@fzaninotto fzaninotto changed the title Telemetry Add domain telemetry on app mount Dec 9, 2020
@fzaninotto fzaninotto added RFR Ready For Review and removed WIP Work In Progress labels Dec 9, 2020
@fzaninotto fzaninotto merged commit 87b2a9a into next Dec 9, 2020
@fzaninotto fzaninotto deleted the telemetry branch December 9, 2020 15:31
@fzaninotto fzaninotto added this to the 3.11 milestone Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants