Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

IndexedDB #6

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

Open
RMo-Sloth wants to merge 24 commits into javascript-tutorial:master
base: master
Choose a base branch
Loading
from RMo-Sloth:translation
Open

Conversation

@RMo-Sloth
Copy link

@RMo-Sloth RMo-Sloth commented Jun 25, 2020

No description provided.

RMo-Sloth added 24 commits June 23, 2020 19:50
@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Jun 25, 2020
Copy link

CLAassistant commented Jun 25, 2020
edited
Loading

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

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

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

@strangernr7 strangernr7 Apr 2, 2021
edited
Loading

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

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

Reviewers

1 more reviewer

@strangernr7 strangernr7 strangernr7 requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

changes requested Waiting for changes and /done from PR author

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /