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: wrong info displayed when quickly navigate from a channel to another #2259

Merged
merged 3 commits into from
May 25, 2022
Merged

Fix: wrong info displayed when quickly navigate from a channel to another #2259

merged 3 commits into from
May 25, 2022

Conversation

hockerschwan
Copy link
Contributor

@hockerschwan hockerschwan commented May 22, 2022

Pull Request Type

  • Bugfix

Related issue
closes #1545

Description
Holds id in getChannelInfoInvidious etc. and compares to this.id, if they are different (route changed), do nothing.

Screenshots (if appropriate)
Before:

ezgif-4-9792565c55.mp4

After:

ezgif-4-5510342c91.mp4

Desktop (please complete the following information):

  • OS: Windows10
  • OS Version: 21H1
  • FreeTube version: 75d59a5

@PrestonN PrestonN enabled auto-merge (squash) May 22, 2022 11:29
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the PR: waiting for review For PRs that are complete, tested, and ready for review label May 22, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented May 22, 2022

This also happens on the local api

FreeTube_VFrY1iewHb.mp4

Would also like to see a fix for that in this PR

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels May 22, 2022
auto-merge was automatically disabled May 22, 2022 12:10

Head branch was pushed to by a user without write access

@PrestonN PrestonN enabled auto-merge (squash) May 22, 2022 12:10
@hockerschwan hockerschwan changed the title Fix: wrong info displayed when quickly navigate from a channel to another (Invidious) Fix: wrong info displayed when quickly navigate from a channel to another May 22, 2022
@hockerschwan
Copy link
Contributor Author

Local api added

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels May 22, 2022
Copy link
Member

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

Choose a reason for hiding this comment

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

LGTM!

src/renderer/views/Channel/Channel.js Outdated Show resolved Hide resolved
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels May 24, 2022
auto-merge was automatically disabled May 24, 2022 15:11

Head branch was pushed to by a user without write access

@PrestonN PrestonN enabled auto-merge (squash) May 24, 2022 15:11
@efb4f5ff-1298-471a-8973-3d47447115dc

@hockerschwan ready for review again?

@hockerschwan
Copy link
Contributor Author

yes please
(changed variable names, no functional changes. )

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels May 24, 2022
Copy link
Member

Choose a reason for hiding this comment

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

LGTM!

@PrestonN PrestonN merged commit a9dc9c0 into FreeTubeApp:development May 25, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label May 25, 2022
@hockerschwan hockerschwan deleted the channel_route_change_inv branch May 25, 2022 13:10
@absidue absidue mentioned this pull request Aug 6, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants