Closed philippe44 closed 3 months ago
Splitting up code is definitely the way to go. I'd probably have called it InfoMenu
, rather than just Info
. In some cases I called it OPML
... because that's what we'd return from them. But it's a bit technical. We could even have Plugins::TIDAL::Menu::Info
, Plugins::TIDAL::Menu::Browse
etc., if we wanted to have a clean structure.
I'm also adding the possibility to add playing track or any track to any given user-modifiable playlist so I'll see if that can be adapted for Tidal.
Thanks!
This PR is a port of what I did on Deezer to handle the M(ore) menu for tracks, albums, artists and playlists, including adding them as a Tidal favorite. I've left the podcast-related code although I'm not sure there is podcast on Tidal or if you plan to support that.
Which might be the most handy to verify updates and you might even want to re-use that in/instead of your getLatestCollectionTimestamp()
I hope this helps and I wanted to do that before you start doing your own so that we can align and maybe share a broader framework for this "More/Favorites management plugin" issue.