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

Fix the Invidious DASH manifest generation #4672

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Feb 12, 2024

Fix the Invidious DASH manifest generation

Pull Request Type

  • Bugfix

Related issue

#4543

Description

Since the YouTube.js upgrade in #4543, the our DASH manifest generation for the Invidious API is broken. Rather ironically I introduced the breaking change on the YouTube.js side, did the YouTube.js upgrade on the FreeTube side and am now the one fixing the issue. Thankfully it's an easy fix.

Screenshots

error-message

Testing

Set the preferred backend to Invidious or turn on proxy through Invidious while the local API is enabled.
You shouldn't get the error in the screenshot anymore.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 30248d6

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 12, 2024 18:49
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 12, 2024
@FreeTubeBot FreeTubeBot merged commit c15f286 into FreeTubeApp:development Feb 15, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 15, 2024
@absidue absidue deleted the fix-invidious-dash branch February 15, 2024 06:32
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Feb 15, 2024
…-user-playlist-2

* development: (66 commits)
  Translated using Weblate (Portuguese (Brazil))
  Translated using Weblate (Czech)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Estonian)
  Translated using Weblate (Turkish)
  Translated using Weblate (Spanish)
  Translated using Weblate (Arabic)
  Translated using Weblate (Italian)
  Translated using Weblate (Polish)
  Translated using Weblate (French)
  Translated using Weblate (Chinese (Traditional))
  Translated using Weblate (Chinese (Simplified))
  Fix overflow issue with share playlist icon drop-down (FreeTubeApp#4677)
  Fix the Invidious DASH manifest generation (FreeTubeApp#4672)
  Hide channel sidebar label under `more` when setting is enabled (FreeTubeApp#4678)
  Translated using Weblate (Norwegian Nynorsk)
  Translated using Weblate (Serbian)
  Remove excessive punctuation as part of distraction settings (FreeTubeApp#4673)
  Bump the stylelint group with 1 update (FreeTubeApp#4669)
  Bump swiper from 11.0.5 to 11.0.6 (FreeTubeApp#4670)
  ...

# Conflicts:
#	src/renderer/components/playlist-info/playlist-info.scss
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.

5 participants