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

Changes in search example #809

Closed
wants to merge 1 commit into from
Closed

Changes in search example #809

wants to merge 1 commit into from

Conversation

shevchuknine
Copy link

No description provided.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Jan 13, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team January 13, 2020 07:55
@@ -493,7 +493,7 @@ books.get('js')
// получить все книги с 'css' < id < 'html'
books.getAll(IDBKeyRange.bound('css', 'html'))

// получить книги с 'html' <= id
// получить книги с 'html' >= id
Copy link
Contributor

@stalanov stalanov Jan 16, 2020

Choose a reason for hiding this comment

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

It's wrong

IDBKeyRange.lowerBound(lower, [open]) означает: >lower (или ≥lower, если open это true)

so we will get books with id higher or equal than lower bound and it should be

// получить книги с id >= 'html'
or
// получить книги с 'html' <= id
like it was

Copy link
Member

Choose a reason for hiding this comment

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

Please take the fix from the English version.

@@ -493,7 +493,7 @@ books.get('js')
// получить все книги с 'css' < id < 'html'
books.getAll(IDBKeyRange.bound('css', 'html'))

// получить книги с 'html' <= id
// получить книги с 'html' >= id
Copy link
Member

Choose a reason for hiding this comment

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

Please take the fix from the English version.

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Jan 19, 2020
@javascript-translate-bot
Copy link
Contributor

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

@CLAassistant
Copy link

CLAassistant commented Jul 24, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Shevchuk Evgeniy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Rnbsov Rnbsov mentioned this pull request May 15, 2022
@ruslauz ruslauz closed this Jun 19, 2022
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.

None yet

6 participants