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

fix(react-native): Replaces the deprecated enableSpotlight option with spotlight #750

Merged
merged 2 commits into from
Jan 7, 2025

Conversation

antonis
Copy link
Contributor

@antonis antonis commented Dec 23, 2024

Fixes getsentry/sentry-react-native#4403

Description

Replaces the deprecated enableSpotlight option with spotlight in the suggested Sentry.init comment.

The option was deprecated in v6.

@krystofwoldrich
Copy link
Member

The E2E tests failures are definitely unrelated of the changes in this PR.

@krystofwoldrich
Copy link
Member

Since this is a comment in the code snippet we can merge it as is, but we should drop support for RN SDK v5 and do a major only supporting RN SDK 6.x.x (not sure about the version exactly).

@krystofwoldrich krystofwoldrich merged commit 6e44204 into getsentry:master Jan 7, 2025
12 of 17 checks passed
@krystofwoldrich
Copy link
Member

Slightly related check for min version #752

PR with change to min 6.0.0 will follow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cli still references invalid enableSpotlight (removed in 6.0.0)
3 participants