Skip to content

Conversation

@marcelklehr
Copy link
Member

@marcelklehr marcelklehr commented Apr 6, 2025

see #1899

@marcelklehr marcelklehr requested a review from Copilot April 6, 2025 09:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

src/lib/IndexedDB.ts:23

  • [nitpick] Consider renaming freeStorageIfNecessary() to cleanupOldLogs() or a similar name that more directly describes its purpose of removing outdated logs.
export async function freeStorageIfNecessary() {

@marcelklehr
Copy link
Member Author

This might even improve sync performance, let's see

@marcelklehr marcelklehr force-pushed the feat/indexeddb-logs branch from 723e4d4 to feff8b2 Compare April 6, 2025 13:53
@marcelklehr marcelklehr merged commit 1b5b97b into develop Apr 9, 2025
30 of 34 checks passed
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.

2 participants