Fix #248: Make analytics opt-in with migration for existing users#433
Open
jloutsch wants to merge 1 commit into
Open
Fix #248: Make analytics opt-in with migration for existing users#433jloutsch wants to merge 1 commit into
jloutsch wants to merge 1 commit into
Conversation
… users Changed default value of enableAnalytics from true to false to address privacy concerns. Previously, analytics were enabled by default without user consent. Added automatic migration that runs on app startup: - Detects existing users with analytics enabled and automatically disables it - New installations use the new default (false) - Users can manually opt-in through Settings → Privacy This is intentional to correct the privacy violation. Users never actively opted in - analytics were enabled by default. The migration ensures all users (new and existing) have analytics disabled by default while still allowing them to opt in if they choose to support the project. Files changed: - app/containers/Settings/reducers.js (line 22) - app/containers/App/index.jsx (lines 132-145 - migration logic) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses issue #248 by making analytics opt-in instead of opt-out. Previously, analytics were enabled by default, which is a privacy concern as users did not explicitly consent to data collection.
Changes
enableAnalyticssetting fromtruetofalsein Settings reducerTesting
Notes
This change affects both new and existing users to ensure consistent privacy-first behavior. Existing users who had analytics enabled will have them automatically disabled, but can easily re-enable if desired through the app settings.
Closes #248