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

Add support for sorting the shorts tab by oldest on the local API #4832

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Mar 31, 2024

Add support for sorting the shorts tab by oldest on the local API

Pull Request Type

  • Feature Implementation

Description

YouTube finally added support for sorting the shorts tab by oldest, which they never added back, unlike the videos and live tabs.
This pull request adds support for that, on the local API only of course, as it might take Invidious a while to catch up (they don't support sorting the live or shorts tabs at all yet).

Testing

Visit a channel with shorts on the local API such as https://www.youtube.com/@GibiASMR/shorts and test that sorting by oldest works and that I didn't break the other sorting options.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: d7b7a40

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 31, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 31, 2024 21:36
@FreeTubeBot FreeTubeBot merged commit f25753c into FreeTubeApp:development Apr 2, 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 Apr 2, 2024
@absidue absidue deleted the shorts-oldest branch April 2, 2024 04:17
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Apr 2, 2024
* development:
  Update icons for add to/remove from quick bookmark button (FreeTubeApp#4825)
  Add support for sorting the shorts tab by oldest on the local API (FreeTubeApp#4832)
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