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

Last.FM integration #658

Closed
rorcores opened this issue Sep 21, 2022 · 2 comments
Closed

Last.FM integration #658

rorcores opened this issue Sep 21, 2022 · 2 comments

Comments

@rorcores
Copy link

I see this has been discussed in the past but dismissed due to feature creep. I understand that but unfortunately a deal breaker for a lot of people. It's such a lightweight and easily toggle-able feature. Any chance we can bring this issue back up?

@martpie
Copy link
Owner

martpie commented Sep 21, 2022

It's such a lightweight and easily toggle-able feature

unfortunately no, it requires opening a browser window to login, registering the app on the file association system so the browser can send the api key back to the app, store the api key in a secure-enough way, etc. So it's a little bit non trivial.

see #231 for more details

@martpie martpie closed this as completed Sep 21, 2022
@rorcores
Copy link
Author

I didn't mean non-trivial from a build perspective, I meant non-trivial from a 'product bloat' perspective. But no worries - I'll try a different product, thanks.

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

No branches or pull requests

2 participants