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.

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

Capability to detect whether a database already exists without creating it #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pbenschop
Copy link
Contributor

Hi Jens, I am with GreatVines in Portland, USA. We used IDBWrapper to create a mock implementation of the Salesforce SmartStore (a database component in the Salesforce Mobile SDK). Since SmartStore allows you to check for the physical presence of a "soup" (represented as a store in IndexedDB) without creating it, we added the capability to the openDB method to abort the transaction if the intent is to only check whether a database exists (without actually creating it). See the added cancelIfNew and CancelMsg options. Maybe this feature could be added to IDBWrapper.

Peter

…ng it

Add cancelIfNew and cancelMsg options to support checking whether a
database exists without creating it.
@jensarps
Copy link
Owner

jensarps commented Feb 3, 2014

Hi @pbenschop,

thanks for all your PRs! Really awesome to see that you're contributing back the changes you made to IDBWrapper. Please give me some time to look through these, that's a lot of code to check out.

Thanks again!

@pbenschop
Copy link
Contributor Author

Hi Jens, thank you for putting this IDBWrapper together in the first place; great piece of work that provide me with good insight how IndexedDB works. In case you decide to accept these changes (which we hope), feel free of course, to change things around if you think there are better ways to achieve the same thing.

@Daisuke-sama
Copy link

Hi guys,
Do you plan to merge that? Or, it has been already implemented somehow?

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.

3 participants