-
Notifications
You must be signed in to change notification settings - Fork 8
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 … … … %xLocale’s appropriate date representation. 08/16/88 (None); 08/16/1988 (en_US); 16.08.1988 (de_DE) %XLocale’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:
isUpcomingin 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 thepremiereTimestampis 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