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

Now will work with default controllerAs (angular 1.5.x default contr… #98

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vlio20
Copy link

@vlio20 vlio20 commented Jul 3, 2016

This module not working if you on angular 1.5 and you are using the module directives inside a component which uses the default controllerAs name which is $ctrl.

This should resolve #96 issue.

@yarl
Copy link

yarl commented Sep 21, 2016

Yes, this one will be helpful, confirmed working on angular 1.5 with $ctrl using this fix.

@vlio20
Copy link
Author

vlio20 commented Oct 24, 2016

@tanquetav , why not merging?

@Ponjimon
Copy link

Pleeease merge this!

@igal1c0de4n
Copy link

+1 would really like to have this (or similar fix) merged
High ROE - small change which makes life easier!

@igal1c0de4n
Copy link

Any updates??

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.

It won't work with default "$ctrl" in component view
4 participants