Kinto / kinto.js

An Offline-First JavaScript Client for Kinto.
http://kintojs.readthedocs.io/
Other
318 stars 72 forks source link

Demo does not work in Microsoft Edge 42.17134.1.0 #869

Open glasserc opened 5 years ago

glasserc commented 5 years ago

Reported by @tiloio originally at https://github.com/Kinto/kinto/issues/1852. On synchronization, we get list() DataError, and on adding a task, we get a DOMException with message DataError.

Because DataError does not occur in our codebase, this is probably a quirk of the Edge IndexedDB implementation. caniuse seems to think that Edge supports most of the IndexedDB v1 spec except for inside "blob web workers", whatever those are. However, getAll is not supported.

@tiloio, any idea if the kinto demo works with version v12.1.1 or earlier? We recently made some changes to use getAll in some places -- maybe we need to add checks around that, or maybe the problem is something deeper like https://github.com/Kinto/kinto.js/blob/master/src/adapters/IDB.js#L50.

dstaley commented 4 years ago

This still occurs in Edge 44.18362.449.0. Since this is the most recent version of pre-Chromium Edge, it might be wise to fix it. However it's my understanding that the Chromium version of Edge is going to roll out to Windows 10 users over the next few weeks. I'm going to try and take a stab at figuring out how to get it working, but based on my quick testing it looks like the developer tools in Edge are seriously lacking.