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 local API about tab for channels without a joined date #4508

Merged

Conversation

absidue
Copy link
Member

@absidue absidue commented Jan 2, 2024

Fix the local API about tab for channels without a joined date

Pull Request Type

  • Bugfix

Description

Currently the local API errors when it tries to extract the about tab for channels that don't have a joined date, like the official @YouTube channel.

Testing

Visit the official @YouTube channel and check that no local API error shows up (the shorts duration parsing error logs will be fixed in a separate pull request)

https://www.youtube.com/channel/UCBR8-60-B28hp2BmDPdntcQ

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.19.1 (nightlies)

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jan 2, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 2, 2024 12:39
Copy link
Collaborator

@kommunarr kommunarr left a comment

Choose a reason for hiding this comment

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

LGTM. Notes for others:

  • You'll need to run yarn again if you can't replicate this issue / see the joined tag at all. Took me a couple of minutes to realize that.
  • This (probably) can't be replaced with just a truthy check because it's using a custom Youtube.js type of Text, not a string.

@FreeTubeBot FreeTubeBot merged commit bb83a9d into FreeTubeApp:development Jan 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 Jan 2, 2024
@absidue absidue deleted the fix-local-about-joined branch January 2, 2024 16:55
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