Skip to content

Conversation

@bjtitus
Copy link
Contributor

@bjtitus bjtitus commented Dec 28, 2025

Android Automattic/pocket-casts-android#4862

Fixes PCIOS-419

When we reorder episodes, a timer is started to sync after 5 seconds. However, if other episodes are being reordered while sync is occurring, the data sets can get out of sync. The fix here is to immediately cancel the timer when dragging begins.

ReorderJump.mov

To test

  • Add many items to the Up Next queue
  • Use Network Link Conditioner to set network profile to 3G
  • Reorder episodes in the Up Next queue using drag and drop in relatively quick succession
  • ✅ Verify that episode maintain ordering

Checklist

  • I have considered if this change warrants user-facing release notes and have added them to CHANGELOG.md if necessary.
  • I have considered adding unit tests for my changes.
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

@bjtitus bjtitus marked this pull request as ready for review December 31, 2025 22:45
@bjtitus bjtitus requested a review from a team as a code owner December 31, 2025 22:45
@bjtitus bjtitus added this to the 8.3 milestone Dec 31, 2025
@bjtitus bjtitus added the Up Next issues related to the Up Next feature label Dec 31, 2025
@bjtitus bjtitus requested a review from SergioEstevao January 2, 2026 01:22
Copy link
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this with the conditioner and order was always kept.
:shipit:

@bjtitus bjtitus merged commit 66d74ee into trunk Jan 5, 2026
9 of 11 checks passed
@bjtitus bjtitus deleted the bjtitus/up-next-reorder-delay branch January 5, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Up Next issues related to the Up Next feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants