search

treojs / treo

Consistent API to IndexedDB
http://treojs.com
MIT License
257 stars 18 forks source link

Updates to websql plugin and shim to use shim in Safari #19

Closed unkillbob closed 9 years ago

unkillbob commented 9 years ago

This is still very much a work in progress as the tests are still broken in Safari (and PhantomJS) primarily due to missing functionality in the shim (can't delete indexes, and IDBTransaction not implementing EventTarget or firing events).

The shim seems quite broken and doesn't look like the github project has had much love in the last few months (not to mention has a pile of open issues/PRs). I take it you're just copying in a dist ver of the shim? Is it ok to edit it like this or should I be opening pull requests against https://github.com/axemclion/IndexedDBShim?

Fixes #17.

alekseykulikov commented 9 years ago

Thanks a lot! It breaks reuse transaction tests, but I will fix them, because I'm going to rewrite this feature for #18 anyway.