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

App crashed when scrolling down in the app's editor #16675

Closed
reginabally opened this issue Jun 1, 2022 · 7 comments
Closed

App crashed when scrolling down in the app's editor #16675

reginabally opened this issue Jun 1, 2022 · 7 comments
Labels
[Deprecated] gutenberg-mobile Deprecated label, use "Gutenberg" instead. [Pri] Medium

Comments

@reginabally
Copy link

reginabally commented Jun 1, 2022

Reported in 5247167-zd-woothemes. The app crashed when the user scrolled down in the app's editor. I can reproduce in SSP with Pixel 4 (emulator), WPAndroid 19.9, and Android 12.

Sentry crash reports can be found here.

Steps to reproduce the behavior

  1. SSP to the account
  2. Go to My Site > Posts > Published, tap the post with the title containing "Crab Cakes Salad"
  3. Scroll down to the bottom of the editor
  4. App crashed

Video recording and more details about the issue (e.g., full post title) can be found in the ticket.

Reported on Samsung SM-G998U, Android 12, WPAndroid 19.9
@khaykov
Copy link
Member

khaykov commented Jun 1, 2022

I think this is one of the occurrences of this issue, but reproducible. In general SIGSEGV issues are very hard to debug and fix, but since we can repro it might be worth giving a try.

Labeling this one as GB, since crash happening in editor.

@khaykov khaykov added [Deprecated] gutenberg-mobile Deprecated label, use "Gutenberg" instead. [Pri] Medium labels Jun 1, 2022
@carl-duncan
Copy link
Contributor

carl-duncan commented Jun 13, 2022

Hey, I will handle this one, just need more information in regards to the ticket.

@hypest
Copy link
Contributor

hypest commented Jun 17, 2022

I see com.swmansion.reanimated being in the stacktrace so, lemme cc @fluiddot since there were more Reanimated lib issues handled recently and this can be relevant.

@hypest hypest added this to Triage in Mobile Gutenberg via automation Jun 17, 2022
@fluiddot
Copy link
Contributor

fluiddot commented Jun 20, 2022

I see com.swmansion.reanimated being in the stacktrace so, lemme cc @fluiddot since there were more Reanimated lib issues handled recently and this can be relevant.

Thanks for the ping @hypest 🙇 !

I checked both the stack trace and user actions and this might be a different problem than the one we're currently trying to fix in (this issue). Most of the crashes we've identified so far are occurring when the editor closes and is re-opened, so they differ from this one. Being this said, I'll cross-reference it with the mentioned issue in case the fixes we apply there also help in addressing this.

@fluiddot
Copy link
Contributor

Based on the steps to reproduce this issue and that the content of the post contains Embed blocks, most likely this is caused by the same problem outlined in this another issue. It has been already solved in WPAndroid 20.3, so we could consider closing it, wdyt?

@fluiddot fluiddot moved this from Triage to Backlog (unscheduled/unassigned) in Mobile Gutenberg Jul 21, 2022
@dcalhoun
Copy link
Member

dcalhoun commented Aug 9, 2022

I agree with @fluiddot. Given the device used is running Android 12 and the affected post content contains an Embed block, it is quite likely this was resolved with WordPress/gutenberg#42517. @reginabally how do you feel about closing this?

@reginabally
Copy link
Author

Thanks for following up, @dcalhoun and @fluiddot! I emailed the user who reported the issue about the fix and asked them if the issue is no longer happening.

I'm closing this for now and will reopen it if needed.

Mobile Gutenberg automation moved this from Backlog (unscheduled/unassigned) to Done (keep clean, manually) Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated] gutenberg-mobile Deprecated label, use "Gutenberg" instead. [Pri] Medium
Projects
Mobile Gutenberg
  
Done (keep clean, manually)
Development

No branches or pull requests

6 participants