feat: add listenbrainz scrobbling support

[CosmicRaptor]

Added listenbrainz scrobbling. (#1029)
ListenBrainz icon was taken from here which is licensed under creative commons.

[CosmicRaptor]

As far as I can tell, I followed their icon design guidelines.
https://github.com/metabrainz/design-system/blob/master/guidelines/design-guidelines.md

[CosmicRaptor]

@Feichtmeier if you want you can replace the PNG with a SVG from the same link for the icon.

[Feichtmeier]

Thank you the code looks very good! GJ!

I just think we might need to call init() again after you activated it or did I miss this?
Like maybe from the app model initListenbrainz() => _exposeService.initListenBrainz()... which calls _listenBrainzService().init() after you entered the api key and after you pressed the button together with the setting you set?

The icon is fine, looks very good!

[CosmicRaptor]

I just think we might need to call init() again after you activated it or did I miss this?

Umm I don't think we need to, the init method simply reads the key from shared prefs and sets up the listenbrainz model. What edge case do you have in mind where this needs to be called twice?

[Feichtmeier]

I just think we might need to call init() again after you activated it or did I miss this?

Umm I don't think we need to, the init method simply reads the key from shared prefs and sets up the listenbrainz model. What edge case do you have in mind where this needs to be called twice?

 ..registerLazySingleton(
      () => ListenBrainzService(
        settingsService: di<SettingsService>(),
      )..init(),
    )

this lazy registration registeres this callback that is called when di<ListenBrainzService>() is called for the first time (in the constructor of th eexposeservice)
and then the address the constructor returtns is saved and is used for later calls of di<ListenBrainzService>()
so this means also init is only called once
so the instance of ListenBrainzService() is created, init() is called right after, once, and thats it :) so we need to call it again if the api key changes from null to sth or from sth to sth else

[CosmicRaptor]

I just think we might need to call init() again after you activated it or did I miss this?

Umm I don't think we need to, the init method simply reads the key from shared prefs and sets up the listenbrainz model. What edge case do you have in mind where this needs to be called twice?

 ..registerLazySingleton(
      () => ListenBrainzService(
        settingsService: di<SettingsService>(),
      )..init(),
    )

this lazy registration registeres this callback that is called when di<ListenBrainzService>() is called for the first time (in the constructor of th eexposeservice) and then the address the constructor returtns is saved and is used for later calls of di<ListenBrainzService>() so this means also init is only called once so the instance of ListenBrainzService() is created, init() is called right after, once, and thats it :) so we need to call it again if the api key changes from null to sth or from sth to sth else

Oh okay I did not know that quirk of get_it! I'll make the changes.

[CosmicRaptor]

I fixed it and as a bonus the app will also update the now playing on listenbrainz.

[Feichtmeier]

Thank you very much! GJ!