Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

Incomplete specification for database creation #42

Closed
pwnall opened this issue Oct 18, 2018 · 2 comments
Closed

Incomplete specification for database creation #42

pwnall opened this issue Oct 18, 2018 · 2 comments

Comments

@pwnall
Copy link

pwnall commented Oct 18, 2018

IIUC, https://domenic.github.io/async-local-storage/#initialize-the-database-promise essentially states that the versionchange handler for the database open request is (event) => event.result.close(). I think the handler needs to create the sole object store used by the DB.

@domenic
Copy link
Collaborator

domenic commented Oct 18, 2018

Does the upgradeneeded handler in step 7 not suffice? It seems to in practice, but the event dance here was pretty complicated...

@pwnall
Copy link
Author

pwnall commented Oct 18, 2018

Oh, sorry, I got confused. The spec is fine as is.

@pwnall pwnall closed this as completed Oct 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants