-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Emby API Key #2117
Emby API Key #2117
Conversation
Add the option to read an API Key from the config file and submit that instead of needing a password. Doesn't remove the use of password so is backwards compatible with users current config file
Thanks for tackling this! 🎉 It looks like there may be some indentation errors in the current commits. You can see the traceback here: https://travis-ci.org/beetbox/beets/jobs/142844029 And the style checker has comments here too: https://travis-ci.org/beetbox/beets/jobs/142844037 Overall, this should be fine if you switch your editor to use 4 spaces for indentation (not real tabs). The Hacking page on the wiki has more details on that. Could you also please add to the documentation to describe how to use this option? (And add a quick note to the changelog?) |
Corrected the use of spaces rather than tabs. Add the option to use an Emby API key to authenticate rather than a password
Awesome! This looks good, and the tests are passing. For the documentation, I also meant to suggest adding to the plugin documentation itself—i.e., |
Added note to the Emby Plugin Documentation
@@ -30,4 +31,5 @@ The available options under the ``emby:`` section are: | |||
- **port**: The Emby server port. | |||
Default: 8096 | |||
- **username**: A username of a Emby user that is allowed to refresh the library. | |||
- **password**: That user's password. | |||
- **apikey**: An Emby API key | |||
- **password**: If no API key is present then enter the user aboves password. |
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.
this needs rewording. Perhaps something like" Emby password (only needed if no API key is provided)
Thanks, @mousey! And I agree, @jrobeson—I've reworded the documentation stuff a bit (and merged). |
Add the option to read an API Key from the config file and submit that
instead of needing a password. Doesn't remove the use of password so is
backwards compatible with users current config file