store.openKeyCursor fallback to store.openCursor

[banjerluke]

Simple fix to support legacy browsers like Edge 18. Fixes #970.

[tofumatt]

Looks good to me, thanks so much!

It's a bit late here so I'll get this merged and deployed sometime tomorrow (Saturday) 🙂

[zspitzer]

is there any chance of this getting merged in?

[leeacker]

also wondering when this might be available - thanks!

[hmhealey]

We're also interested in having this in as well. We're having to temporarily patch this for now.

[zspitzer]

This problem is made worse coz I found in my use case the old edge javascript engine wasn't handling the exception like other browsers and my it wasn't reaching my automated error reporting.

[davidbgk]

Same here, we had to downgrade to 1.7.3 to continue being Edge-compatible.

[kjcpaas]

We have also downgraded to an older version. Will keep watch on this PR.

Thanks for working on it! 😄

[tobloef]

Seems like the Travis CI timed out. @tofumatt, can we maybe re-run it, so we can get this merged? If it's a bug in the PR, I will happily fix it if I can get a little more info on what's going wrong.

[ronnix]

Is there a way we can help move this forward? @banjerluke maybe you can rebase your fork on the current master branch to resolve the conflicts? Hopefully Travis will then successfully run the tests 🤞

[banjerluke]

OK, rebased onto master...