FreeTube
FreeTube copied to clipboard
Add support for channel's `Courses` tab
Add support for channel's Courses tab
Pull Request Type
- [x] Bugfix - fixes some issues with podcasts and releases as well
- [x] Feature Implementation - add support for channel's
Coursestab
Related issue
https://github.com/iv-org/invidious/issues/5144
Dependent on: https://github.com/LuanRT/YouTube.js/pull/883 https://github.com/iv-org/invidious/pull/5158
Description
This PR adds support for the courses tab on the channel page for both local api and Invidious API
Screenshots
Testing
test 1
- enter this url https://www.youtube.com/@InfinityLearn_NEET/courses
- be navigated to courses tab
test 2
- enter this url https://www.youtube.com/@InfinityLearn_NEET/
- switch to courses tab
test 3
- hide courses tab in distraction free settings
- go to https://www.youtube.com/@InfinityLearn_NEET
- don't see courses tab
Desktop
- OS: Fedora Linux
- OS Version: 41
- FreeTube version: latest nightly
As a YouTube.js update with the required changes was released and is being used on the development branch, what would you like to do with this pull request? Wait until the changes on the Invidious side are merged or proceed with a local API implementation first and follow up with an Invidious implementation once the relevant changes have landed on their side?
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
Error with IV API when navigating towards this courses playlist VirtualBoxVM_2Zd9PiFHIR.mp4
Error with the Local API when starting to watch the courses playlist VirtualBoxVM_8WkCfT3lLT.mp4
The error with invidious api is a bug in invidious (seems to be because you have to pay to watch one of the videos). See: https://inv.nadeko.net/playlist?list=PLmdFyQYShrjfOiKmTtgQFsdorN-0Aq2uJ in a browser
The local api error is because we have the same video showing up multiple times in the playlist
(both of these issues can happen outside of this PR)