Skip to content

Default params argument of bindAll to empty object #167

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
Closed
elado opened this issue Sep 12, 2014 · 1 comment
Closed

Default params argument of bindAll to empty object #167

elado opened this issue Sep 12, 2014 · 1 comment
Assignees
@jmdobry
Labels
done feature
Milestone
1.0.0-rc.2

Comments

@elado
Copy link

elado commented Sep 12, 2014 

Article.bindAll($scope, 'articles');

currently throws an error, it requires another empty {} argument for params, even if I want all items.
@jmdobry
Copy link
Member

jmdobry commented Sep 12, 2014

Easy enough

@jmdobry jmdobry added this to the 1.0.0-rc.2 milestone Sep 12, 2014
@jmdobry jmdobry self-assigned this Sep 12, 2014
@jmdobry jmdobry added in progress and removed ready labels Sep 13, 2014
jmdobry added a commit to js-data/js-data that referenced this issue Sep 13, 2014
@jmdobry
Backport js-data/js-data-angular#145.
af1faf8 
Backport js-data/js-data-angular#156.
Backport js-data/js-data-angular#167.
@jmdobry jmdobry added done and removed in progress labels Sep 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmdobry jmdobry

Labels
done feature
Projects
None yet
Milestone
1.0.0-rc.2
Development

No branches or pull requests
2 participants
@elado @jmdobry