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

Support directory items and plugin contents rather than just videos #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

krruzic
Copy link

@krruzic krruzic commented Aug 22, 2023

No description provided.

@krruzic
Copy link
Author

krruzic commented Aug 23, 2023

This PR techincally turns this addon into maybe more than was originally intended. My interest in it is basically as a better favouriting system and now with this that's what it does. I can group any content together into one list and activate a plugin, a video, music or whatever else.

Issues:

  • you can add the shortlist items to a shortlist, causing weird issues, or even add a shortlist directory, causing recursion. I don't know kodi addons enough to figure out how to detect this and hide the context menu items in that case [SOLVED]

@Nanomani
Copy link

Nanomani commented Aug 28, 2023

Hi,

thanks for your code :)
I have make an update for add more information filled :

GIT : master...Nanomani:plugin.program.shortlist:extend-type-item-and-more-info-filled-
ZIP: https://github.com/Nanomani/plugin.program.shortlist/archive/refs/heads/extend-type-item-and-more-info-filled-.zip

Other code is not from me. It is from "Sekt0r" in Kodi forum.
Source : https://forum.kodi.tv/showthread.php?tid=370104&pid=3148252#pid3148252

So I have no credit for that ...

PS : in a perfect world, it would be great to have a setting to store db files on a shared space on a NAS...

@krruzic
Copy link
Author

krruzic commented Aug 28, 2023

I was going to add that code too I was looking at that thread yesterday. I'm not sure about storing it remotely like on a NAS but maybe some kind of solution like that would be possible.

@Nanomani Nanomani mentioned this pull request Sep 11, 2023
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.

2 participants