Introduce OfflineTrack for use with OfflinePlaylist#93
Conversation
a in memory Track representation.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
pSpitzner
left a comment
There was a problem hiding this comment.
Looks good to me.
Im still wondering how this helps with our desired synced-playlists-history class (that serves as ground truth, where we keep track of two remote playlists, and have for each track on each side a time stemp when it was added/moved/deleted)
but thats a topic for a pen-and papter discussion.
I ran into issues with |
This pull request introduces the new
OfflineTrackclass to provide an in-memory representation of tracks, similar to the existingOfflinePlaylist. This allows us to preserve track information even after deletion from the remote service, and to keep the Playlist class hierarchy cleanly separated between in-memory and service-synced implementations.