Skip to content

Fix playback identity and episode resume state#147

Merged
kalzEOS merged 1 commit into
mainfrom
fix/episodes-representations
May 17, 2026
Merged

Fix playback identity and episode resume state#147
kalzEOS merged 1 commit into
mainfrom
fix/episodes-representations

Conversation

@kalzEOS
Copy link
Copy Markdown
Owner

@kalzEOS kalzEOS commented May 17, 2026

Summary

  • Canonicalize VOD playback identity across launch points
  • Fix auto-next and episode state updates for series playback
  • Add per-episode VOD progress bars and safe subtitle state handling
  • Keep Continue Watching collapsed to one series-level entry

Verification

  • rtk ./gradlew :app:compileDebugKotlin

Notes

  • This branch was reviewed for playback identity regressions before release.

@kalzEOS kalzEOS merged commit 422d37f into main May 17, 2026
1 check passed
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a2e7753f76

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +8844 to 8847
val stableParentId = resumeItem.parentSeriesId?.takeUnless { it.isBlank() }
if (!stableParentId.isNullOrBlank()) {
contentRepository.findSeriesItemById(stableParentId, authConfig)
} else {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Fall back when parent-ID lookup misses

When parentSeriesId is present, this path only calls findSeriesItemById, which reads the cached SERIES index and can return null when that index is missing/stale (it does not fetch remotely). In that case the series card cannot open and the UI falls into the “Unable to open series details” path, even though a title-based search could still resolve the parent. This regresses first-open behavior for continue-watching series episodes that have parentSeriesId but no local index yet.

Useful? React with 👍 / 👎.

Comment on lines +8757 to +8758
?: resolvedParentsSnapshot[entry.key]?.streamId?.takeUnless { it.isBlank() }
?: resolvedParentsSnapshot[entry.key]?.id?.takeUnless { it.isBlank() }
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use matching keys for resolved-parent grouping

displayGroupingKey looks up resolvedParentsSnapshot[entry.key], but resolvedParents is populated with displayEntries keys (groupKey, e.g. series:...), not raw repository keys. Because these key spaces differ, the lookup never hits, so resolved parents are never applied during regrouping and legacy series episodes without embedded parent metadata remain split into multiple cards instead of collapsing.

Useful? React with 👍 / 👎.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant