Skip to content

Fix premieres on the subscription page#3240

Merged
FreeTubeBot merged 2 commits intoFreeTubeApp:developmentfrom
absidue:fix-premieres-subscription
Mar 5, 2023
Merged

Fix premieres on the subscription page#3240
FreeTubeBot merged 2 commits intoFreeTubeApp:developmentfrom
absidue:fix-premieres-subscription

Conversation

@absidue
Copy link
Member

@absidue absidue commented Mar 2, 2023

Fix premieres on the subscription page

Pull Request Type

  • Bugfix

Description

Currently if you are subscribed to a channel that has a premiere video, when you refresch your subscriptions on the local API without RSS, you will see that it says it was uploaded x hours ago and an error in the console.

This pull request fixes it.

Screenshots

error

Testing

Subscribe to https://www.youtube.com/@onetv4548 and refresh your subscriptions with the local API without using RSS.

Desktop

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

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 2, 2023 18:26
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 2, 2023
PikachuEXE
PikachuEXE previously approved these changes Mar 3, 2023
@absidue absidue marked this pull request as draft March 3, 2023 06:57
auto-merge was automatically disabled March 3, 2023 06:57

Pull request was converted to draft

@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 3, 2023
@absidue
Copy link
Member Author

absidue commented Mar 3, 2023

Looks like the premiere date is shown incorrectly after loading from the subscriptions cache, I want to fix that too, so I've converted this to a draft, so that it doesn't accidentally get merged before that happens.

@absidue absidue added the PR: WIP label Mar 3, 2023
@absidue absidue marked this pull request as ready for review March 3, 2023 19:00
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 3, 2023 19:00
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 3, 2023
@absidue absidue removed the PR: WIP label Mar 3, 2023
Copy link
Member

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc left a comment

Choose a reason for hiding this comment

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

it fixes it but there seems to be an bug that needs to be addressed. Should i create issue for them?

edit: dismiss the issue shown related to #3242 but do note the issue on the second half of the video

VirtualBoxVM_huFoQ7Xff0.mp4

@absidue
Copy link
Member Author

absidue commented Mar 3, 2023

pretty sure everything later on in the video is because of #3242 and subscribing to an errored channel (not sure why it's even showing the subscription button)

@FreeTubeBot FreeTubeBot merged commit 667ee83 into FreeTubeApp:development Mar 5, 2023
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 5, 2023
@absidue absidue deleted the fix-premieres-subscription branch March 5, 2023 19:29
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