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

replace native `bind` with amp-bind #53

Merged
merged 1 commit into from Feb 24, 2015

Conversation

Projects
None yet
4 participants
@mmacaula
Contributor

mmacaula commented Feb 20, 2015

A small change to make life easier for those using phantom.js with ampersand collections. Fixes #52. Threw in a test just to make sure I didn't break anything. It passed before the change as well of course.

@kamilogorek

This comment has been minimized.

Show comment
Hide comment
@kamilogorek

kamilogorek Feb 20, 2015

Member

Looks fine!

Member

kamilogorek commented Feb 20, 2015

Looks fine!

@wraithgar

This comment has been minimized.

Show comment
Hide comment
@wraithgar
Member

wraithgar commented Feb 24, 2015

+1

wraithgar added a commit that referenced this pull request Feb 24, 2015

Merge pull request #53 from mmacaula/master
replace native `bind` with amp-bind

@wraithgar wraithgar merged commit b590ba9 into AmpersandJS:master Feb 24, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@wraithgar

This comment has been minimized.

Show comment
Hide comment
@wraithgar

wraithgar Feb 24, 2015

Member

v1.4.1

Member

wraithgar commented Feb 24, 2015

v1.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment