search

inflatablefriends / lastfm

Portable .Net library for Last.fm
Other
100 stars 61 forks source link

Add method Artist.GetTopAlbumsByMbidAsync #129

Closed mew22 closed 1 year ago

mew22 commented 5 years ago

Is it possible to add this method, based on the existing Artist.GetTopAlbumsAsync but with Mbid ?

rikkit commented 5 years ago

Hi @mew22, if the API will accept this parameter, it's definitely possible to add. Unfortunately I've got very little free time at the moment to implement it for you, but if there is a pull request, I'll make time to merge it in and release. Thanks

klinge commented 5 years ago

Is it ok if I have a go at implementing this?

rikkit commented 5 years ago

@klinge go for it :)

klinge commented 5 years ago

I'll start with trying to get tests to run on .Net Core/Linux. If I fix that then I will continue with this one.

GmausDev commented 1 year ago

Its still open?

rikkit commented 1 year ago

@GmausDev yep!

GmausDev commented 1 year ago

@rikkit i looking at #141 PR and its look like done.?

rikkit commented 1 year ago

Ah, sorry. Yes it looks like it is complete.