-
Notifications
You must be signed in to change notification settings - Fork 57
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: Opening New or Existing Empty Blog Post #2440
Comments
The console log showed this error when I tried to replicate the issue E/WordPress-API: An error occurred while fetching History revisions Could this be a possible reason of the crash? |
I tried the testing steps using WPAndroid 14.7-rc-3 on Pixel 3 Android 10 and was unable to get the crash to happen. I labeled the issue as Note: seems to match up with this crash report from Sentry: Sentry Issue: WORDPRESS-ANDROID-7MW
|
It seems pretty unlikely that is the cause, but the additional info is helpful. Thanks for reporting @gibtang ! Just confirming, but am I understanding correctly that this only happens with a new or empty post? So you can open a non-empty post with no issues? cc: @theck13 |
May be some relevant discussion here: facebook/react-native#25490 |
@mchowning, I can't reproduce the crash with WordPress Android 14.7-rc-3. |
99 events have been tracked for crash WORDPRESS-ANDROID-7MW in 14.7 and 14.7.1 since they were released 15d and 4d ago on May 15 and May 4 respectively. Events: 511 |
Events in the last 90d: 575 |
Events in the last 90d: 1,000 Events in the last 90d seem to be trending up. |
This issue looks totally related to the RN integration of the block editor so, I'm moving it to the gutenberg-mobile repo. |
I've set the "[Pri]" label to "Medium" as it looks like the issue is happening to very specific devices, mostly on TECNO devices. |
I've attempted to reproduce this issue using a local build of WPAndroid from the
And didn't have any luck in reproducing it until I tried it on my LG G5, Android 8 by opening an existing post. Something special to note here, the app did not crash. I just happened to notice the error message in the logs:
I went back through my other test devices above - this time looking not for a crash, but for this error message and sure enough, I get it on every device, but only during the very first load of a post. I'll see the SoLoader error, but it's not fatal and it doesn't cause the app to crash. Looking at the Sentry Crash Event for this issue, it appears the highest version of WPAndroid that actually crashed was I've set the Sentry event as "resolved as of 17.7" so we can track if there is a regression. @hypest do you think I should leave this ticket open for monitoring or just close it? |
Quick update The sentry ticket was resolved by @AmandaRiu. I am going to close this ticket with the assumption that Sentry will highlight it again if the issue resurfaces as a crash in future releases.
|
Expected
Opening a new or existing blog post that is empty will not crash.
Observed
Opening a new or existing blog post that is empty will crash. See the stack trace below for details.
Stack Trace
Reproduced
Tested
Google Pixel 2 on Android 10.0 with WordPress Android alpha-213
The text was updated successfully, but these errors were encountered: