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

[plugin.video.invidious] 0.2.2+nexus.0 #4363

Merged
merged 1 commit into from Sep 11, 2023

Conversation

petterreinholdtsen
Copy link

@petterreinholdtsen petterreinholdtsen commented Aug 8, 2023

A privacy-friendly way of watching YouTube content. Uses the great Invidious web service's API to do the heavy lifting.

Third nexus repository release from new repository.

Checklist:

  • My code follows the add-on rules and piracy stance of this project.
  • I have read the CONTRIBUTING document
  • Each add-on submission should be a single commit with using the following style: [plugin.video.foo] v1.0.0

@petterreinholdtsen petterreinholdtsen changed the title [plugin.video.invidoius] 0.2.0+nexus.0 [plugin.video.invidoius] 0.2.1+nexus.0 Aug 10, 2023
@ramiro
Copy link

ramiro commented Aug 10, 2023

@petterreinholdtsen there is a typo in the PR title invidoius -> invidious

@petterreinholdtsen petterreinholdtsen changed the title [plugin.video.invidoius] 0.2.1+nexus.0 [plugin.video.invidious] 0.2.1+nexus.0 Aug 11, 2023
@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 11, 2023 via email

@petterreinholdtsen petterreinholdtsen changed the title [plugin.video.invidious] 0.2.1+nexus.0 [plugin.video.invidious] 0.2.2+nexus.0 Aug 11, 2023
@Lunatixz Lunatixz self-assigned this Aug 15, 2023
@Lunatixz Lunatixz self-requested a review August 15, 2023 15:31
Copy link

@Lunatixz Lunatixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your submission, please address the issues and we can proceed.

plugin.video.invidious/addon.xml Outdated Show resolved Hide resolved
plugin.video.invidious/addon.xml Show resolved Hide resolved
@@ -0,0 +1,4 @@
# Update translations when the english master changes

update-po:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this file from the PR request.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem. Why do you want to hide the recipe on how to update the translations, build a distribution zip and make releases?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you want to hide the recipe on how to update the translations, build a distribution zip and make releases?

I guess they have their own way to do releases.
I always branch out and create PRs from the branch, and keep my own potential none desired custom scripts to my own master branch.

(Btw, I was the origin for the Makefile. I use it for maintaning addon releases in my own kodi repo.)

plugin.video.invidious/resources/lib/invidious_plugin.py Outdated Show resolved Hide resolved
plugin.video.invidious/resources/lib/invidious_plugin.py Outdated Show resolved Hide resolved
petterreinholdtsen added a commit to petterreinholdtsen/kodi-invidious-plugin that referenced this pull request Aug 22, 2023
@petterreinholdtsen petterreinholdtsen force-pushed the invidious-0.2.0+nexus.0 branch 2 times, most recently from fc0e9fb to 9210762 Compare August 24, 2023 20:54
@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 26, 2023 via email

@Lunatixz
Copy link

Lunatixz commented Aug 26, 2023

@petterreinholdtsen Lets start with your addon.xml... below is how it should appear on your end.... Make sure you have the dependency installed to Kodi (its available in the official repository).

    <requires>
        <import addon="xbmc.python" version="3.0.0"/>
        <import addon="script.module.requests" version="2.22.0"/>
        <import addon="script.module.inputstreamhelper" version="0.5.2"/>
        <import addon="script.module.infotagger" version="0.0.5" />
    </requires>

add below to invidious_plugin.py
from infotagger.listitem import ListItemInfoTag

at lines below change list_item.setInfo("video", {'title':'kilroy'}) to

info_tag = ListItemInfoTag(list_item, 'video')
info_tag.set_info({'title':'kilroy'})

https://github.com/xbmc/repo-plugins/pull/4363/files#diff-70e6e459897139bee66316b0f474d4e178c727fa69d348201e6d843f646bffffR135

https://github.com/xbmc/repo-plugins/pull/4363/files#diff-70e6e459897139bee66316b0f474d4e178c727fa69d348201e6d843f646bffffR225

@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 30, 2023 via email

@Lunatixz
Copy link

I'm not sure why the dependency isn't installing... its live in the repo: http://mirrors.kodi.tv/stats?downloadstats&filter=script.module.infotagger

I would manually download infotaggers zip and install it...

@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 30, 2023 via email

@Lunatixz
Copy link

Lunatixz commented Aug 30, 2023

@petterreinholdtsen Great, glad you worked it out... only one issue remaining. The fanart is not 1920x1080,
ERROR: fanart should have either 1280x720 or 1920x1080 or 3840x2160 but it has 1080x540

@Lunatixz Lunatixz self-requested a review August 30, 2023 20:43
@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 31, 2023 via email

@Lunatixz
Copy link

png or jpg.

A privacy-friendly way of watching YouTube content. Uses the great Invidious
web service's API to do the heavy lifting.

Nexus repository release from new repository.
@petterreinholdtsen
Copy link
Author

petterreinholdtsen commented Aug 31, 2023 via email

@petterreinholdtsen
Copy link
Author

Is there anything more for me to do to get this into wider distribution?

Copy link

@romanvm romanvm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Lunatixz Lunatixz merged commit fd435d0 into xbmc:nexus Sep 11, 2023
1 check passed
@petterreinholdtsen petterreinholdtsen deleted the invidious-0.2.0+nexus.0 branch January 19, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants