-
-
Notifications
You must be signed in to change notification settings - Fork 931
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 no telemetry option and improve analytics code clarity #2740
base: main
Are you sure you want to change the base?
Conversation
…` instead of `initPlausible` for improved clarity
if any people reviewing source for their own verification of its purpose, this should make it much clearer what this does
changing the option in the Select dropdown breaks things right now, gotta figure out why
… to submitPlausibleEvent
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
Just need to polish UI for this and should be good to go after! |
Need to change cypress onboarding test here to add the new telemetry option and rename the element ids: spacedrive/apps/web/cypress/e2e/1-onboarding.spec.cy.ts Lines 133 to 139 in a0173ca
|
Also just remembered I need to figure out how to properly migrate the old telemetry preference to this new property format |
This PR is primarily aimed at adding a "none" option for analytics which has been highly requested, with some other related changes to help improve clarity of some of our analytics functionality for people who may be reviewing the code for their own peace of mind.