Skip to content

The current example for skip fails when the pageNumber is 0 #1491

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

Closed
wants to merge 1 commit into from

Conversation

leopic
Copy link
Contributor

@leopic leopic commented Dec 20, 2013

When pageNumber is 0, the current example -1 times pageSize which can result in negative value and yield the following error: !ERROR stack MongoError: bad skip value in query

@tychoish
Copy link
Contributor

tychoish commented May 9, 2014

Thanks for the contribution! First of all, sorry for the delay in getting back to you on this. I'd like to get this merged in an published, but first we need you to sign the MongoDB Contributor Agreement. Once we have the signature on file we can move forward.

Thanks so much for your effort, and sorry again for the delay!

Cheers,
sam

@leopic
Copy link
Contributor Author

leopic commented May 9, 2014

Hey Sam

All set, signed under Leo Picado O.

On Fri, May 9, 2014 at 9:59 AM, Sam Kleinman [email protected]:

Thanks for the contribution! First of all, sorry for the delay in getting
back to you on this. I'd like to get this merged in an published, but first
we need you to sign the MongoDB Contributor Agreementhttp://www.mongodb.com/legal/contributor-agreement.
Once we have the signature on file we can move forward.

Thanks so much for your effort, and sorry again for the delay!

Cheers,
sam


Reply to this email directly or view it on GitHubhttps://github.com//pull/1491#issuecomment-42682368
.

Leo Picado, UI Guy.
www.leonardopicado.com | [email protected]

@kay-kim
Copy link
Contributor

kay-kim commented May 9, 2014

Merged. Thanks again.

Regards,

Kay Kim

@kay-kim kay-kim closed this May 9, 2014
mongo-cr-bot pushed a commit that referenced this pull request Jul 28, 2022
* DOCSP-21630 Consistent Syntax Examples: S-W

* internal review feedback

* addressing additional internal review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants