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

Safari Technical Preview 74 Drops Web SQL #115

Closed
neofuture opened this issue Jan 30, 2019 · 4 comments

Comments

Projects
None yet
3 participants
@neofuture
Copy link

commented Jan 30, 2019

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@neofuture

This comment has been minimized.

Copy link
Author

commented Jan 30, 2019

In the new technical preview for safari, web sql has been removed, hence me finding your solution, but your solution falls back to openDatabase which causes a error when using PERM

@bkniffler

This comment has been minimized.

Copy link
Contributor

commented Jan 30, 2019

I guess this is the line causing the issue:

if (isSafari) {

@neofuture, you can also just use "IDB" as mode instead of "PERM".

@neofuture

This comment has been minimized.

Copy link
Author

commented Jan 30, 2019

Indeed, I was just saying that its broken in tech preview

@ClickSimply

This comment has been minimized.

Copy link
Owner

commented Jan 31, 2019

2.1.3 now does feature detection even if it's running in Safari, so PERM should work in Safari 74.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.