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

Cleanup unused code in the store #5776

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

absidue
Copy link
Member

@absidue absidue commented Sep 30, 2024

Cleanup unused code in the store

Pull Request Type

  • Code cleanup

Description

  • The last{Video,Short,Live,Community}RefreshTimestampByProfile state entries have been unused since Implement persistent subscription cache #5185.
  • The get{Video,Short,Live,Community}Cache getters have never been used.
  • The getCurrentVolume getter seems to have been unused ever since its addition to the code base in 77e7168 and the state property that it references has never existed from what I can tell.

Desktop

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

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 30, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) September 30, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: waiting for review For PRs that are complete, tested, and ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants