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.

Sign up for GitHub

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

Jump to bottom

MS Edge 18 Object doesn't support property or method 'openKeyCursor' #970

Open
becvert opened this issue Jun 15, 2020 · 3 comments · May be fixed by #971
Open

MS Edge 18 Object doesn't support property or method 'openKeyCursor' #970

becvert opened this issue Jun 15, 2020 · 3 comments · May be fixed by #971
Labels
bug

Comments

@becvert
Copy link

becvert commented Jun 15, 2020

Hello,
Just letting you know that this error happens since PR #874
@tofumatt tofumatt added the bug label Jun 15, 2020
@tofumatt
Copy link
Member

Ah, seems like the answer here might be to try using openKeyCursor and falling back to openCursor to maintain browser compatibility. I'd happily review a PR with a fix, but if not I'll try to get to this soon, sometime this weekend I imagine. Thanks for reporting it!

banjerluke added a commit to banjerluke/localForage that referenced this issue Jun 18, 2020
@banjerluke
store.openKeyCursor fallback to store.openCursor
bcf654d 
To support legacy browsers like Edge 18. Fixes localForage#970.
@banjerluke banjerluke linked a pull request Jun 18, 2020 that will close this issue
Open
@leeacker
Copy link

leeacker commented Aug 4, 2020

Should this be in the latest version? I am still experiencing it with 1.9.0

ronnix added a commit to Delegation-numerique-en-sante/mesconseilscovid that referenced this issue Aug 24, 2020
@ronnix
Tech: revient à une version de localforage qui marche avec IE et Edge
2a5fb55 
Le problème a été introduit dans la 1.7.4 : localForage/localForage#970

Un correctif est en attente d’intégration : localForage/localForage#971
ronnix added a commit to Delegation-numerique-en-sante/mesconseilscovid that referenced this issue Aug 24, 2020
@ronnix
Tech: revient à une version de localforage qui marche avec IE et Edge
b3adb08 
Le problème a été introduit dans la 1.7.4 : localForage/localForage#970

Un correctif est en attente d’intégration : localForage/localForage#971
@ronnix ronnix mentioned this issue Aug 24, 2020
Merged
ronnix added a commit to Delegation-numerique-en-sante/mesconseilscovid that referenced this issue Aug 24, 2020
@ronnix
Tech: revient à une version de localforage qui marche avec IE et Edge
96797b3 
Le problème a été introduit dans la 1.7.4 : localForage/localForage#970

Un correctif est en attente d’intégration : localForage/localForage#971
@tfrisvold
Copy link

I am seeing this in version 1.9 too.. backed off to 1.7.3.. but I would like to see this in a new release so I can keep current with this package. Any ETA on a new version with this fix?

banjerluke added a commit to banjerluke/localForage that referenced this issue Aug 24, 2021
@banjerluke
store.openKeyCursor fallback to store.openCursor
ba06361 
To support legacy browsers like Edge 18. Fixes localForage#970.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

store.openKeyCursor fallback to store.openCursor banjerluke/localForage
4 participants
@tofumatt @becvert @leeacker @tfrisvold