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.

Sign up for GitHub

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

Jump to bottom

Misuse of bind method - change all .bind to .apply #80

Open
taylorjdawson opened this issue Nov 27, 2019 · 0 comments
Open

Misuse of bind method - change all .bind to .apply #80

taylorjdawson opened this issue Nov 27, 2019 · 0 comments
Assignees
@taylorjdawson
Labels
Refactor
Milestone
Web3data.js Versi...

Comments

@taylorjdawson
Copy link
Contributor

Use Case:

Users want code that is compliant with JS standards! Wait who are we kidding JS has no standards! Either way we are misusing bind it should be apply to add the this context to a method.

Requirements:

  • Change all occurrences of .bind to .apply

Acceptance Criteria:

-All tests passing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@taylorjdawson taylorjdawson

Labels
Refactor
Projects
None yet
Milestone
Web3data.js Version 1! 🙌
Development

No branches or pull requests
1 participant
@taylorjdawson