Open
Conversation
strangernr7
suggested changes
Apr 2, 2021
strangernr7
left a comment
There was a problem hiding this comment.
Don't have the time to finish, but I noticed a couple of changes needed
| - Supports key range queries, indexes. | ||
| - Can store much bigger volumes of data than `localStorage`. | ||
| - Slaat bijna elke soort waarde per key op, er zijn meerdere types key nodig. | ||
| - Ondersteund transacties voor betrouwbaarheid. |
There was a problem hiding this comment.
Suggested change
| - Ondersteund transacties voor betrouwbaarheid. | |
| - Ondersteunt transacties voor betrouwbaarheid. |
You had ondersteund here but ondersteunt below
| - Slaat bijna elke soort waarde per key op, er zijn meerdere types key nodig. | ||
| - Ondersteund transacties voor betrouwbaarheid. | ||
| - Ondersteunt queries van key ranges en indexen. | ||
| - Kan een groter volumes data opslaan dan `localStorage`. |
There was a problem hiding this comment.
Suggested change
| - Kan een groter volumes data opslaan dan `localStorage`. | |
| - Kan veel grotere hoeveelheden gegevens opslaan dan `localStorage`. |
| - Kan een groter volumes data opslaan dan `localStorage`. | ||
|
|
||
| That power is usually excessive for traditional client-server apps. IndexedDB is intended for offline apps, to be combined with ServiceWorkers and other technologies. | ||
| Deze mogelijkheden zijn normaal gesproken excessief voor traditionele client-server apps. IndexedDB is bedoeld voor offline apps, om gecombineerd te worden met met ServiceWorkers en andere technologiën. |
There was a problem hiding this comment.
Suggested change
| Deze mogelijkheden zijn normaal gesproken excessief voor traditionele client-server apps. IndexedDB is bedoeld voor offline apps, om gecombineerd te worden met met ServiceWorkers en andere technologiën. | |
| Deze mogelijkheden zijn normaal gesproken excessief voor traditionele client-server apps. IndexedDB is bedoeld voor offline apps, om gecombineerd te worden met ServiceWorkers en andere technologiën. |
| Deze mogelijkheden zijn normaal gesproken excessief voor traditionele client-server apps. IndexedDB is bedoeld voor offline apps, om gecombineerd te worden met met ServiceWorkers en andere technologiën. | ||
|
|
||
| The native interface to IndexedDB, described in the specification <https://www.w3.org/TR/IndexedDB>, is event-based. | ||
|
|
There was a problem hiding this comment.
Suggested change
I'm pretty sure this is an extra line
| De functie geeft een `openRequest` object, we kunnen naar de events in dit object luisteren: | ||
| - `success`: de database is gereed, er is een "database object" in `openRequest.result`, die we kunnen gebruiken voor verdere functie invocaties. | ||
| - `error`: openen van de database is gefaald. | ||
| - `upgradeneeded`: de database is gereed, maar de versie is verouderd ( zie onderstaand ) |
There was a problem hiding this comment.
Suggested change
| - `upgradeneeded`: de database is gereed, maar de versie is verouderd ( zie onderstaand ) | |
| - `upgradeneeded`: de database is gereed, maar de versie is verouderd (zie onderstaand) |
Nitpick
| - `upgradeneeded`: de database is gereed, maar de versie is verouderd ( zie onderstaand ) | ||
|
|
||
| **IndexedDB has a built-in mechanism of "schema versioning", absent in server-side databases.** | ||
| **IndexedDB heeft een ingebouwd mechanisme van "schema versies", in tegenstelling tot server-side databases** |
There was a problem hiding this comment.
Suggested change
| **IndexedDB heeft een ingebouwd mechanisme van "schema versies", in tegenstelling tot server-side databases** | |
| **IndexedDB heeft een ingebouwd mechanisme van "schema versies", in tegenstelling tot server-side databases.** |
Again, nitpick
|
Please make the requested changes. After it, add a comment "/done". |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.