Details
Details
- Reviewers
michal tomek - Commits
- rCOMMe8645d4fe752: [web-db] add indexedDB config
Run this code:
try { const value = await localforage.setItem('test', { prop: 'test prop', }); console.log(value); } catch (err) { console.log(err); }
and check in devtools (web and desktop) if database was properly created
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- db-on-web-2
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/database/worker/web-db-worker.js | ||
---|---|---|
12 | Is this version used e.g. for migrations? |
web/database/worker/web-db-worker.js | ||
---|---|---|
12 | It can be, but we won't use it for this purpose. For database (SQLite) versioning, we will use the same pattern as on native (PRAGMA user_version;). IndexedDB will be a key-value store with two keys and I don't think it'll ever need to migrate - if so I would introduce the keyserver's approach with an additional metadata key. Updating this version here will result in calling native callback upgradeneeded, but it's not that good, and locaforage as well as other similar libs does not supper this. |