Movatterモバイル変換


[0]ホーム

URL:


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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
RMo-Sloth wants to merge24 commits intojavascript-tutorial:master
base:master
Choose a base branch
Loading
fromRMo-Sloth:translation

Conversation

RMo-Sloth
Copy link

No description provided.

@javascript-translate-botjavascript-translate-bot added the review neededReview needed, please approve or request changes labelJun 25, 2020
@javascript-translate-botjavascript-translate-bot requested a review froma teamJune 25, 2020 22:08
@CLAassistant
Copy link

CLAassistant commentedJun 25, 2020
edited
Loading

CLA assistant check
All committers have signed the CLA.

@iliakaniliakan mentioned this pull requestJun 25, 2020
Copy link

@strangernr7strangernr7 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 meerderetypes key nodig.
-Ondersteund transacties voor betrouwbaarheid.
-Ondersteunt queries vankeyranges en indexen.
-Kan een grotervolumes dataopslaan dan `localStorage`.

Choose a reason for hiding this comment

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

Suggested change
- Kaneen groter volumes data opslaan dan`localStorage`.
- Kanveel grotere hoeveelheden gegevens opslaan dan`localStorage`.


That power is usually excessive for traditionalclient-server apps. IndexedDB isintended for offline apps,to be combined withServiceWorkersand other technologies.
Deze mogelijkheden zijn normaal gesproken excessief voor traditioneleclient-server apps. IndexedDB isbedoeld voor offline apps,om gecombineerd te worden met metServiceWorkersen 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 metmetServiceWorkers 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, wekunnen naar de eventsin dit object luisteren:
- `success`:dedatabase isgereed, er is een"database object" in `openRequest.result`,die wekunnen gebruiken voor verdere functie invocaties.
- `error`:openen van de database is gefaald.
- `upgradeneeded`:dedatabase isgereed, maar de versie isverouderd ( 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


**IndexedDBhas a built-in mechanism of "schemaversioning",absentin server-side databases.**
**IndexedDBheeft een ingebouwd mechanisme van "schemaversies", integenstelling totserver-side databases**
Copy link

@strangernr7strangernr7Apr 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-botjavascript-translate-bot removed the review neededReview needed, please approve or request changes labelApr 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-botjavascript-translate-bot added the changes requestedWaiting for changes and /done from PR author labelApr 2, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@strangernr7strangernr7strangernr7 requested changes

Assignees
No one assigned
Labels
changes requestedWaiting for changes and /done from PR author
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@RMo-Sloth@CLAassistant@javascript-translate-bot@strangernr7

[8]ページ先頭

©2009-2025 Movatter.jp