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

IndexedDB #6

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Conversation

RMo-Sloth
Copy link

No description provided.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Jun 25, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team June 25, 2020 22:08
@CLAassistant
Copy link

CLAassistant commented Jun 25, 2020

CLA assistant check
All committers have signed the CLA.

Copy link

@strangernr7 strangernr7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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`.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Kan een groter volumes data opslaan dan `localStorage`.
- Kan veel grotere hoeveelheden gegevens 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.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.


The native interface to IndexedDB, described in the specification <https://www.w3.org/TR/IndexedDB>, is event-based.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 )

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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


**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**
Copy link

@strangernr7 strangernr7 Apr 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@javascript-translate-bot javascript-translate-bot removed the review needed Review needed, please approve or request changes label Apr 2, 2021
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@javascript-translate-bot javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Waiting for changes and /done from PR author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants