librespot
librespot copied to clipboard
Support additionnal SpotifyUri types
Is your feature request related to a problem? Please describe.
Currently, only a subset of URI are supported, while some other like genre or medialistconfig aren't, which preventing deserializing data coming from certain endpoint like rootlist
Describe the solution you'd like
SpotifyId could be refactored to aligned better with the underlying core concept; this way, SpotifyId and SpotifyItemType would become SpotifyUri, and the enum would define sub part, such as SpotifyID (Spotify ID in the above doc), but not exclusively, like it is the case with the current structure.
Describe alternatives you've considered
Alternatively, I have currently extended the existing Struct/Enum relation, which allows supporting these additional URI type in a non-breaking way, but this leads to inconsistency, which is likely to grow as more URI type are added
Additional context
N/A
Happy to help implementing the later if that is of interest
Sounds like a good idea. Feel free to open a PR!