Skip to content

Prevent crashes when retained posts are deleted - #26015

Open
crazytonyli wants to merge 2 commits into
trunkfrom
bugfix/sentry-post-lifetime
Open

crazytonyli wants to merge 2 commits into
trunkfrom
bugfix/sentry-post-lifetime

Conversation

@crazytonyli

@crazytonyli crazytonyli commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

The app may crash when delete/trash an post that's already deleted on the server but yet to be synced.

Here is the easiest way to reproduce the crash. It may not exactly match what's happening to the real users, but the root cause is the same:

  1. Simulate a slow network, go to the app's posts list. The posts are being refreshed in the background, slowly.
  2. Tap more actions on a post, and select Trash. You'll see an alert. Leave the alert on.
  3. On the web, delete the selected post.
  4. The app's post list is refreshed.
  5. Confirm the trash post alert. App crashes.

Sentry records a new issue for this crash (i.e., https://a8c.sentry.io/issues/7703437016/?project=1438083), but it's not actually new (i.e., https://a8c.sentry.io/issues/7398708809/?project=1438083).

@crazytonyli
crazytonyli marked this pull request as ready for review September 14, 2026 06:59
@crazytonyli crazytonyli added this to the 27.3 milestone Sep 14, 2026
@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number34454
VersionPR #26015
Bundle IDorg.wordpress.alpha
Commitfad6452
Installation URL1hvhavh9bo048
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number34454
VersionPR #26015
Bundle IDcom.jetpack.alpha
Commitfad6452
Installation URL26o1cd6hplolg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants