Skip to content

YOUTUBE SHORTS#105

Open
PranAD-dev wants to merge 4 commits intoSCE-Development:devfrom
PranAD-dev:dev
Open

YOUTUBE SHORTS#105
PranAD-dev wants to merge 4 commits intoSCE-Development:devfrom
PranAD-dev:dev

Conversation

@PranAD-dev
Copy link
Copy Markdown
Contributor

Youtube shorts work now, even shorts in a playlist work, shorts look more vertical than normal videos cuz their shorts

@PranAD-dev
Copy link
Copy Markdown
Contributor Author

disregard what i said in commit #19df74d playlist containing shorts in playlists do work.

@MatthewTran22
Copy link
Copy Markdown
Contributor

disregard what i said in commit #19df74d playlist containing shorts in playlists do work.

@PranAD-dev Individual Youtube Shorts work. I noticed that for shorts playlists I had to refresh VLC a lot to see the next video, could just be me but can you verify on your end?

@vineeshah vineeshah closed this Apr 11, 2026
@vineeshah vineeshah reopened this Apr 11, 2026
Comment thread modules/cache.py
@@ -156,5 +166,9 @@ def write_cache(self):
@staticmethod
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[MEDIUM] The parse_qs function returns a dictionary where keys might be missing if the query parameter is not present. Accessing ['v'][0] directly without checking for the existence of the 'v' key can lead to a KeyError if the URL is malformed or does not contain the 'v' parameter. Consider adding a check for the 'v' key before accessing it, or using .get('v') with a default value and then handling the potential None.

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.

3 participants