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.

Sign up for GitHub

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

Jump to bottom

Replace DexieJS with native IndexedDB interfaces. #164

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Replace DexieJS with native IndexedDB interfaces. #164

merged 2 commits into from
Oct 10, 2024

Conversation

samcf
Copy link
Owner

@samcf samcf commented Oct 10, 2024 •
edited
Loading

Resolves #163 and #162

  • Remove DexieJS as a dependency (~100kb, 10%)
  • New idb.cljs namespace provides context and hooks for reading and writing to IndexedDB
  • Implements import and export feature using MessagePack
  • Removes storage.cljs, refactoring listeners to state.cljs or idb.cljs
  • Some formatting changes

samcf added 2 commits October 8, 2024 18:41
@samcf
Replace DexieJS with native usage of IDB.
bcad3c2 
New hooks for interacting with IDB.
Replace callback properties with `addEventListener`.
Some config changes.
@samcf
Implement import/export using MessagePack.
1c8f93a 
Refactor some provider listeners.
Some minor formatting changes.
@samcf samcf self-assigned this Oct 10, 2024
@samcf samcf merged commit 56f3974 into main Oct 10, 2024
@samcf samcf deleted the replace-dexiejs branch October 10, 2024 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@samcf samcf

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Replace DexieJS with native IndexedDB interface
1 participant
@samcf