search

expo / sentry-expo

MIT License
202 stars 83 forks source link

Warning error when running dev app #355

Closed RSNFreud closed 6 months ago

RSNFreud commented 1 year ago

Summary

image

I get this sent every time the app reloads

Managed or bare workflow? If you have ios/ or android/ directories in your project, the answer is bare!

managed

What platform(s) does this occur on?

Android

SDK Version (managed workflow only)

6.2.0

Environment

expo-env-info 1.0.5 environment info: System: OS: Windows 10 10.0.22621 Binaries: Node: 16.18.0 - C:\Program Files\nodejs\node.EXE Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD npm: 8.1.1 - ~\AppData\Roaming\npm\npm.CMD IDEs: Android Studio: AI-212.5712.43.2112.8609683 npmPackages: @expo/webpack-config: ^18.0.4 => 18.0.4 expo: ~48.0.16 => 48.0.16 react: 18.2.0 => 18.2.0 react-dom: 18.2.0 => 18.2.0 react-native: 0.71.7 => 0.71.7 react-native-web: ~0.18.10 => 0.18.12 Expo Workflow: managed

Reproducible demo or steps to reproduce from a blank project

-

alessandrocapra commented 1 year ago

To add to this, I confirm I have installed the following packages following the installation guide for sentry-expo:

"sentry-expo": "~6.2.0",
"@sentry/react-native": "4.15.2",
github-actions[bot] commented 10 months ago

This issue is stale because it has been open for 60 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

RSNFreud commented 10 months ago

Not stale.

On Fri, 8 Sept 2023, 12:06 github-actions[bot], @.***> wrote:

This issue is stale because it has been open for 60 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

— Reply to this email directly, view it on GitHub https://github.com/expo/sentry-expo/issues/355#issuecomment-1711334905, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFQIF2XFLXAJM36JSIEA2W3XZLNX5ANCNFSM6AAAAAA2DDTJSM . You are receiving this because you authored the thread.Message ID: @.***>

krystofwoldrich commented 10 months ago

@RSNFreud Thank you for the message.

It looks like you have Sentry disabled for a development environment. What is the expected behavior? Only the first log message and no working?

Or is this a mistake and you enabled Sentry for dev?

RSNFreud commented 10 months ago

The expected behaviour is for it not to show that error every time the app is rerendered

krystofwoldrich commented 10 months ago

@RSNFreud Got it, we will take a look if we can make the log less intrusive.

github-actions[bot] commented 8 months ago

This issue is stale because it has been open for 60 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

RSNFreud commented 8 months ago

Not stale

On Sun, 3 Dec 2023 at 11:05, github-actions[bot] @.***> wrote:

This issue is stale because it has been open for 60 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

— Reply to this email directly, view it on GitHub https://github.com/expo/sentry-expo/issues/355#issuecomment-1837416585, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFQIF2SZAPYIVRD4GBIACEDYHQ6G7AVCNFSM6AAAAAA2DDTJSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZXGQYTMNJYGU . You are receiving this because you were mentioned.Message ID: @.***>

krystofwoldrich commented 6 months ago

Hi @RSNFreud , sentry-expo was recently deprecated in favor of @sentry/react-native which supports Expo out of the box now.

And this warning is not present in the @sentry/react-native. We reworked the Expo integration and the logs and warnings are now much more pleasant and less intrusive for Expo Go users.

https://github.com/getsentry/sentry-react-native/pull/3466