Adds support for Invidious, aimed at the lekma/video.plugin.invidious version.
The version of Invidious currently in the Kodi repos is unmaintained and the public GitHub repo was archived.
Updates the README and addon description to mention Invidious.
Makes YouTube an optional addon, and adds Invidious as an optional addon.
On the side I did a few minor refactors.
Documentation
Documentation for variables can be done by doing the docstring syntax under it.
Now that the comment was moved to a docstring, we get hints when hovering over variables.
Types
Normally types are lowercase, i.e. list[str], not List[str] afaik.
Notes
#get_video_id is no longer YouTube specific. Should we consider moving it to another file?
The failed to parse notification payload message will always go through when watching on Invidious. I'm not sure how to work with this since I don't have the YouTube app setup to review existing behavior or understand the purpose tbh.
Adds support for Invidious, aimed at the lekma/video.plugin.invidious version. The version of Invidious currently in the Kodi repos is unmaintained and the public GitHub repo was archived.
On the side I did a few minor refactors.
Documentation
Documentation for variables can be done by doing the docstring syntax under it.
Types
Normally types are lowercase, i.e.
list[str], notList[str]afaik.Notes
#get_video_idis no longer YouTube specific. Should we consider moving it to another file?Related