Automattic / Automattic/pocket-casts-android

UpNextQueue should not expose the Playable object

Open
#84 0 comments 0 reactions 0 assignees View on GitHub
[Type] Tech Debt Migrated
Dominant language
Kotlin
Stars
2.8k
Forks
308
Avg merge
1d 10h
Merged PRs (30d)
187

Description

From the original issue report:

> Doing this makes it possible for somewhere else to change the episode's properties. Exposing a uuid would be fine or expose an automatically updating version.

Contributor guide

Open the contributing guide

Research direction

Start by locating UpNextQueue and all call sites that access its Playable object. Review how callers use the episode properties and compare the issue's suggested UUID or automatically updating version. Done means the queue no longer exposes a mutable Playable object while callers retain the required episode identity or updates; the issue names no tests or files to run.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.