-
Notifications
You must be signed in to change notification settings - Fork 207
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
Add IndexedDB getAllEntries() explainer #868
base: main
Are you sure you want to change the base?
Conversation
```js | ||
dictionary IDBGetAllEntriesOptions { | ||
any query = null; // A key or an `IDBKeyRange` identifying the records to retrieve. | ||
[EnforceRange] unsigned long count; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be helpful to add a short inline comment // number of records to retrieve.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. Done!
|
||
[Exposed=(Window,Worker)] | ||
partial interface IDBObjectStore { | ||
// After the `getAllEntries()` request completes, the `IDBRequest::result` property |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might have to define Exceptions; maybe say it is a same with GetAll, GetAllKeys or any new exception?
This PR adds the explainer for the IndexedDB getAllEntries() operation, which is proposed by the following W3C IndexedDB issue:
w3c/IndexedDB#206