-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix: mark premiere videos as unplayable #96
base: master
Are you sure you want to change the base?
Conversation
subplot.append(localizedString(30055)) | ||
|
||
if self.premiereTimestamp: | ||
subplot.append(localizedString(30063) + self.premiereTimestamp.strftime("%x %X")) |
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.
Context:
Directive Meaning Example … … … %x
Locale’s appropriate date representation. 08/16/88 (None); 08/16/1988 (en_US); 16.08.1988 (de_DE) %X
Locale’s appropriate time representation. 21:30:00 (en_US); 21:30:00 (de_DE) — Excerpt from Python documentation for format codes.
is this still relevant? |
what happens if you click on an unplayable item that is not a directory? |
I still experience this even with the latest version, so it should still be relevant. It loads for a moment as if it was going to play the video, then does nothing and returns to the menu I was originally on.
At the time I wrote this PR (but I haven't tested this more recently), it does nothing. (No sound, no action, it's as if the click wasn't registered.) Later I'll rebase this PR and try to record a video with a before vs after. Though atm I'm having some issues with Invidious (even at home on my private self-hosted instance) so we'll see if I'll actually be able to test it. 🤔 |
this is strange because if i click right now on an upcoming stream (v3.2.6) i get an error notification telling me "this live event will begin in 10 hours" could you give me a videoId to test with by the way no need to rebase the pr ( a lot has changed) i'll introduce this from scratch |
there was a small bug in v3.2.5 that prevented the errors from the instance to be notified |
also sorry for not coming back to you earlier (the last two years irl have been...hectic for me) |
Huh, I'll make sure I actually am on the latest version and try again tonight. Could be I still have the fork with some of my PRs applied. I'll try to reproduce this somewhere this week if not tonight and come back to you!
No worries, I get it! There's no rush to these things, and worst case scenario, I can just update the plugin locally, perks of open-source. 👍🏽 |
Previously, if a channel, search result, or feed, featured a premiered video that was still upcoming, the Invidious add-on would show it like any other video, but would throw errors when trying to play it.
There was no indication in the UI that it's a premiered video.
This PR makes three changes:
isUpcoming
in the API, it makes the video unplayable.premiereTimestamp
, display that in the subplot instead of the view count and publish date.Screenshots
Notes
__date__
(which afaik delegates to__transform__
internally) because thepremiereTimestamp
is a conditional attribute. script.module.iapc was throwing exceptions for videos that didn't have it, so I opted to handle transforming it in this plugin.References