-
Notifications
You must be signed in to change notification settings - Fork 401
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
Crash reports from production #147
Comments
I have the error too
|
The stack trace seems to suggest that the issue is coming from when react is unmounting some components - makes me think if it's related to showing a toast and then immediately going to another screen and thus unmounting the screen responsible for showing the toast? |
Is it the same as Can you guys try install the master version which includes a fix to the problem above
|
I don't think it's the same issue, since we're not getting keyboard related exceptions. Since I can't repro locally, the only way for me to test it out with the master version is to release to production - is master stable for me to do that? Cheers |
We released an update that cuts back on where we use the toast messages - now they're only used in certain places, e.g. when you're interacting with the app. But we're still seeing the same volume of crash reports on this (about 20 per day). The only thing that I can think could be related is that in some places we show a toast, and then navigate immediately somewhere else, I wonder if that could be causing some issues... |
Are you using the master version? |
I'm still on the latest version from npm |
I'm having this issue. And to reproduce, it's pretty simple actually: Pressing the android back button to "leave" the app does it for me. (So you press the back button when you're at the top level of your navigation stack if you have one) |
Sure. Just tried, still happening unfortunately. |
I can not reproduce. Can anyone provide a minimal reproducible demo? (use |
Same here! This is my unobfuscated log and it's happening only in production and it started happening after upgrading react-native:
This line does reference the keyboard:
@mattwoberts I think your issue is related to the keyboard even though you are not picking up any keyboard related exceptions. @sunnylqm this is fixed in v3.3.0? |
Sorry for the late response @wmonecke . I'm not sure if it's related to the keyboard or not. We've just moved to app center for our app, which collects crash data. On the iOS, there's hardly anything relating to ToastContainer, but on the android side, it's happening all the time: |
how about change this.keyboardListener?.remove?.(); |
First of all, thanks for making and supporting this library :)
We're using it in our app (V 3.2.1), and we're seeing quite a lot of errors from production (from the Google play console). Here's a typical stack trace:
We're totally unable to recreate this internally. I think it's happening from our app when we have code that responds to a notification being received in the foreground - in that case it uses this lib to show a message to the user. The toast is wrapped in this helper:
So the addition of the try/catch doesn't catch anything, I'm a bit stumped about where to look next...
Any ideas?
The text was updated successfully, but these errors were encountered: