-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Officially support React #40
Comments
Is this on the roadmap, does it only work with angular right now? |
@davidemerritt Yep this is a roadmap item. It officially only works with angular, however in theory React should already work just fine and you can pass |
I have it working as you described but wondering what will break... Thanks for the update. |
Works fine in production import * as Sentry from '@sentry/capacitor'
import { init as sentryReactInit } from '@sentry/react'
Sentry.init({
dsn: "__DSN__",
}, sentryReactInit) |
Currently implementing sentry with Example of what I have below import * as Sentry from '@sentry/capacitor'
import * as SentryVue from '@sentry/vue'
Sentry.init({
app,
dsn: "__DSN__",
integrations: [new BrowserTracing()] // <- TS error
}, SentryVue.init) |
@MaxWeisen npm install --save-exact @sentry/[email protected] For explaination see getsentry/sentry-javascript#4474 (comment) |
Yes, for the time being, sentry/capacitor is not compatible with the newer versions of Sentry/vue, react, angular (7.x). |
This works. Thank you for posting! |
Any update on this? |
Currently the Capacitor SDK is working just fine with React. |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
Add an example app, and fully test all features.
The text was updated successfully, but these errors were encountered: