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: Update history_utils.dart #575

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

Conversation

nope3472
Copy link
Contributor

PR Description

Add your description

Key Changes:
Moved History Cleanup to a Background Isolate:
Introduced clearExpiredHistoryAsync() in history_utils.dart, which utilizes compute() to filter expired history entries in the background.
Added _filterExpiredHistory(), a top-level function that runs in the isolate to remove expired data efficiently.
Resolves Issue #551

  • Closes #

Checklist

  • [x ] I have gone through the contributing guide
  • [ x] I have updated my branch and synced it with project main branch before making this PR
  • [ x] I am using the latest Flutter stable branch (run flutter upgrade and verify)
  • [ x] I have run the tests (flutter test) and all tests are passing

Added/updated tests?

We encourage you to add relevant test cases.

  • Yes
  • [x ] No, and this is why: please replace this line with details on why tests have not been included

OS on which you have developed and tested the feature?

  • [x ] Windows
  • macOS
  • [ x] Linux

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.

1 participant