Automattic / Automattic/pocket-casts-android
UpNextQueue should not expose the Playable object
Open
[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
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