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

fix error arguments #1097

Merged
merged 4 commits into from
Jul 27, 2021
Merged

Conversation

kaznovac
Copy link
Contributor

  • error constructor takes 0..1 arguments
  • use style from
    if (onItem.length > 2) {
    throw Error(
    `The \`onItem\` callback function passed to autoPagingEach must accept at most two arguments; got ${onItem}`
    );
    }

@CLAassistant
Copy link

CLAassistant commented Dec 25, 2020

CLA assistant check
All committers have signed the CLA.

@kaznovac kaznovac changed the title Fix error arguments fix error arguments Dec 25, 2020
@remi-stripe
Copy link
Contributor

@kaznovac Thanks for opening a PR to fix this! We'll have a look though it will likely be after the holidays!

@richardm-stripe richardm-stripe merged commit 1e3ce8f into stripe:master Jul 27, 2021
@kaznovac kaznovac deleted the fix-error-arguments branch July 27, 2021 12:51
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.

4 participants