Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add LastFM to the store #59

Merged
merged 8 commits into from Feb 7, 2022
Merged

Add LastFM to the store #59

merged 8 commits into from Feb 7, 2022

Conversation

Saiyato
Copy link
Contributor

@Saiyato Saiyato commented Jan 21, 2022

Initial upload of LastFM to the Volumio 3 store.

Initial upload of LastFM to the Volumio 3 store.
To adhere to the new standard
Fix category (miscellanea doesn't exist anymore)
Fix syntax error
To meet Volumio3 requirements
Tune node dep
1. Fix move to new category
2. Fix crashes when used unauthenticated
@volumio
Copy link
Owner

volumio commented Feb 7, 2022

Excellent.
Why dont you ship it with an Api key for the plugin? This will make it much easier for users to use it

@volumio volumio merged commit 27fede7 into volumio:master Feb 7, 2022
@volumio
Copy link
Owner

volumio commented Feb 7, 2022

One more thing: the icon for the service is not right. Please update it according to the other icons examples

@Saiyato
Copy link
Contributor Author

Saiyato commented Feb 8, 2022

Hi @volumio,

  1. I don't ship it with my key, because then I also need to provide my own secret and it would be linked to my account. Apart from not feeling comfortable sharing that information, I think a new version should have auth happening using the Volumio auth-service. What is needed to do this?

  2. What is wrong with the service icon? It's "fa-lastfm", and the svg-file is only used for albumart (in the browse-area). I can't find any guidelines as to what kind of icon should be used instead.

PS: I did see a typo in index.js ("fa fa-lastfm" instead of "fa-lastfm"); but it translates to the same.

  "volumio_info": {
    "prettyName": "LastFM Scrobbler",
    "icon": "fa-lastfm",

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants