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

Update TileManager.ts #320

Closed

Conversation

drodriguezs-minsait
Copy link

Fix error: DOMException: Failed to execute 'put' on 'IDBObjectStore': The object store uses out-of-line keys and has no key generator and the key parameter was not provided.

Fix error: DOMException: Failed to execute 'put' on 'IDBObjectStore': The object store uses out-of-line keys and has no key generator and the key parameter was not provided.
@drodriguezs-minsait
Copy link
Author

IDB PUT expects a third parameter with the key when saving the tile.

@allartk
Copy link
Owner

allartk commented Oct 17, 2023

Hi, thanks for the merge request.
How do you call the saveTile method to get the error. If I see correctly (on mobile..) key is optional in the idb put method.

Other things: You added the dist folder, but those are in gitignore since they are a build result. There is also a linting error.

@drodriguezs-minsait
Copy link
Author

Not in all browsers, but in the latest version of Chrome, I get this error:

image

To reproduce the error, it is necessary to start from a clean database (I have previously deleted the database).

@allartk
Copy link
Owner

allartk commented Oct 23, 2023

I'm not able to repeat it on

Please provide more info, like versions and a code example.

@allartk
Copy link
Owner

allartk commented Oct 23, 2023

BTW Your screenshot shows the exact opposite of the error you reported at first...

@allartk
Copy link
Owner

allartk commented Nov 1, 2023

Closed because inactivity

@allartk allartk closed this Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants