tuxfoo / jellyfin-skill

This skill allows audio playback from a Jellyfin server
Other
8 stars 4 forks source link

Feature Request - Ability to play a genre #11

Open drgnomage opened 3 years ago

drgnomage commented 3 years ago

As title, I'm not sure if this is doable with the current Jellyfin API.

tuxfoo commented 3 years ago

Thanks for the feature requests, I plan to go back to playing with Mycroft soon.

drgnomage commented 3 years ago

Let me know if you need a beta tester!

tuxfoo commented 3 years ago

I have added this feature to the dev branch https://github.com/tuxfoo/jellyfin-skill/tree/dev-branch

drgnomage commented 3 years ago

I can't seem to specify the branch when using mycroft-msm install so I checked out the branch in the /opt/mycroft/skills/jellyfin-skill.tuxfoo directory. Is that the correct way of changing the branch or have I missed something?

tuxfoo commented 3 years ago

I can't seem to specify the branch when using mycroft-msm install so I checked out the branch in the /opt/mycroft/skills/jellyfin-skill.tuxfoo directory. Is that the correct way of changing the branch or have I missed something?

checkout should work, I think you need to do git pull after checkout

drgnomage commented 3 years ago

Doesn't seem to like loading it. It shows as red when I list skills in the mycroft cli and won't become active.

08:15:33.099 | ERROR | 1401 | mycroft.skills.intent_service:handle_converse_error:193 | jellyfin-skill.tuxfoo: converse requested but skill not loaded

(.venv) pi@picroft:/opt/mycroft/skills/jellyfin-skill.tuxfoo $ git status On branch dev-branch Your branch is up to date with 'origin/dev-branch'.

nothing to commit, working tree clean (.venv) pi@picroft:/opt/mycroft/skills/jellyfin-skill.tuxfoo $ git pull Already up to date. (.venv) pi@picroft:/opt/mycroft/skills/jellyfin-skill.tuxfoo $

git show: commit 2d204ac6fb1c7e27243be2787e93dcb7d3bff25d (HEAD -> dev-branch, origin/dev-branch) Author: tuxfoo tuxfoo@librem.one Date: Mon Oct 25 11:11:09 2021 +1300

Play a genre

diff --git a/README.md b/README.md index 74791cd..5d1d5a3 100644 --- a/README.md +++ b/README.md @@ -23,6 +23,7 @@ This skill supports the common play framework! This means you don't have to spec

skill.log: 2021-10-30 08:02:49.430 | ERROR | 1401 | msm.mycroft_skills_manager | Error running install_or_update on jellyfin-skill.tuxfoo: MergeConflict