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

Migrate Trending to the composition API #6397

Merged

Conversation

absidue
Copy link
Member

@absidue absidue commented Dec 16, 2024

Migrate Trending to the composition API

Pull Request Type

  • Refactoring - Composition API migration

Description

This pull request migrate the Trending component to the composition API.

Testing

  1. Test that the trending page loads data with both API backends
  2. Test that refreshing works
  3. Test that it correctly restores from the cache (leave the trending page and come back again)

Desktop

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

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 16, 2024 20:42
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 16, 2024
@absidue absidue removed the request for review from kommunarr December 16, 2024 22:35
@efb4f5ff-1298-471a-8973-3d47447115dc

If i manually refresh one feed the others will also refresh when i navigate to them. This makes testcase 2 harder to test

VirtualBoxVM_XkeT9hh1aV.mp4

@absidue
Copy link
Member Author

absidue commented Dec 17, 2024

That is the same as it was before. I probably should have specified in the pull request description that this pull request just migrated the component to the composition API and didn't try to fix any bugs (like other pull requests have in the past), so this time you are mainly testing that it works the same as on the development branch.

@FreeTubeBot FreeTubeBot merged commit ff15d83 into FreeTubeApp:development Dec 18, 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 Dec 18, 2024
@absidue absidue deleted the trending-composition-api branch December 18, 2024 05:11
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