Skip to content

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

Open
vlio20 wants to merge 1 commit into
kamilkp:masterfrom
vlio20:master
Open

Now will work with default controllerAs (angular 1.5.x default contr…#98
vlio20 wants to merge 1 commit into
kamilkp:masterfrom
vlio20:master

Conversation

@vlio20

@vlio20 vlio20 commented Jul 3, 2016

Copy link
Copy Markdown

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

yarl commented Sep 21, 2016

Copy link
Copy Markdown

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

@vlio20

vlio20 commented Oct 24, 2016

Copy link
Copy Markdown
Author

@tanquetav , why not merging?

@Ponjimon

Copy link
Copy Markdown

Pleeease merge this!

@igalgh

igalgh commented Apr 23, 2017

Copy link
Copy Markdown

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

@igalgh

igalgh commented Mar 26, 2018

Copy link
Copy Markdown

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