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

fixed wrong calculated pagination size. #628

Merged
merged 3 commits into from
Aug 30, 2022
Merged

fixed wrong calculated pagination size. #628

merged 3 commits into from
Aug 30, 2022

Conversation

b4rtaz
Copy link
Collaborator

@b4rtaz b4rtaz commented Aug 30, 2022


name: fixed wrong calculated pagination size.

New Pull Request

Checklist

  • I am not disclosing a vulnerability.
  • My code is conform the code style
  • I have made corresponding changes to the documentation
  • I have updated Typescript definitions when needed

@changeset-bot
Copy link

changeset-bot bot commented Aug 30, 2022

🦋 Changeset detected

Latest commit: 12d5bb1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@moralisweb3/evm-api Patch
@moralisweb3/evm-utils Patch
moralis Patch
@moralisweb3/core Patch
@moralisweb3/auth Patch
@moralisweb3/api-utils Patch
@moralisweb3/sol-utils Patch
@moralisweb3/sol-api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@b4rtaz b4rtaz requested a review from ErnoW August 30, 2022 10:41
@b4rtaz b4rtaz enabled auto-merge (squash) August 30, 2022 10:46
}

next = () => {
if (!this._nextCall) {
public hasNext = () => {
Copy link
Member

Choose a reason for hiding this comment

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

Can you add tsdocs comment to public methods?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yep.

throw new MoralisApiError({
code: ApiErrorCode.PAGE_LIMIT_EXCEEDED,
message: 'Cannot call .next(). Page limit exceeded.',
message:
'Page limit exceeded! Before call this method check an existence of the next page by .hasNext() method.',
Copy link
Member

Choose a reason for hiding this comment

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

👍

@b4rtaz b4rtaz requested a review from ErnoW August 30, 2022 11:11
@b4rtaz b4rtaz merged commit a6b30fc into main Aug 30, 2022
@b4rtaz b4rtaz deleted the feat/pagination branch August 30, 2022 11:34
@github-actions github-actions bot mentioned this pull request Aug 30, 2022
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.

2 participants