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
LB-303: Add instruction to add last.fm api key to custom_config.py to developement environment setup guide #349
Conversation
docs/dev/devel-env.rst
Outdated
LASTFM_API_URL = "https://ws.audioscrobbler.com/2.0/" | ||
LASTFM_API_KEY = "USE_LASTFM_API_KEY" | ||
|
||
Update the last.fm api key with your key. After doing this, your |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should add a explanation on where to get this key from.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just adding link to API form would be enough or should i add something else too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can obtain a Last.FM API key from here: <link>
docs/dev/devel-env.rst
Outdated
@@ -111,6 +111,17 @@ Update the strings with your client ID and secret. After doing this, your | |||
ListenBrainz development environment is able to authenticate and log in from | |||
your MusicBrainz login. | |||
|
|||
Also, in order for last.fm import to work, you should also update your last.fm api key in this file. Look for the following section in the file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We try to write Last.FM
instead of last.fm
in documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok 👍
@mayhem, you should take a look at the verbiage here once, IMO. |
It doesn't flow super well, but really, there is nothing here stopping this from being merged. |
Summary
There was no instruction given for adding last.fm api to custom_config.py file in the development environment setup guide.
Problem
Solution
Added the instructions to the guide.