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

IndexedDB fixes #28

Merged

sgwilym merged 3 commits into main from idb-fixes

Sep 10, 2024

Contributor

sgwilym commented Sep 10, 2024

Calls to KvBatch.commit were not being bound properly when used in a prefixed driver.
Calling a cursor with prevunique / nextunique caused problems in Safari because these are only allowed on indexes. We don't need it, so just use prev / next, respectively.
An invalid IndexedDB bound was being created in certain list operations with a prefix supplied. My fix is to use a closed upper bound if the start and end of the bound are the same.

sgwilym added 2 commits

September 10, 2024 16:10


          Bind KV.batch commit properly

992d4ff


          IndexedDB: Fix cursor direction + fix creation of invalid bounds

213deda

sgwilym added the bug label


          Really fix IDB bound creation

787aa26

sgwilym merged commit 94c9cbc into main

1 check passed

sgwilym deleted the idb-fixes branch

September 10, 2024 15:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug