Re: [w3c/IndexedDB] Add getAllRecords() method and update getAll()/getAllKeys() to support direction option (PR #461)

@SteveBeckerMSFT commented on this pull request.



> @@ -1371,6 +1371,38 @@ To <dfn>convert a value to a key range</dfn> with
 
 </div>
 
+
+A <dfn>potentially valid key range</dfn> is a value that is a [=key/type|type of key=] or a [=key range=].

Yes, you're right in pointing out that this abuses the definition of key type.  I've updated the definition in a new attempt.  Perhaps we need to update the definition of key to better support the getAll()/getAllKeys() dictionary overload, but I'd like to avoid this larger change if possible.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/IndexedDB/pull/461#discussion_r2170105628
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/IndexedDB/pull/461/review/2963821929@github.com>