Skip to content

Align with latest NG 1.5 changes #7

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

Merged
merged 2 commits into from
Jan 9, 2016
Merged

Conversation

borislit
Copy link
Contributor

@borislit borislit commented Jan 9, 2016

Set $ctrl as default controllerAs param and removed option to configure scope (set it isolated by default).

All this in order to be compliant with latest changes to angular 1.5:
angular/angular.js#13710

…re scope (set it isolated by default).

All this in order to be complient with latest changes to angular 1.5:
angular/angular.js#13710
@toddmotto
Copy link
Owner

Awesome, would you be able to run npm i and then gulp to generate the /dist files please, we could bump this to 0.0.3 inside package.json too? :)

@borislit
Copy link
Contributor Author

borislit commented Jan 9, 2016

@toddmotto Sure. Ill push in few minutes

…re scope (set it isolated by default).

All this in order to be complient with latest changes to angular 1.5:
angular/angular.js#13710
@toddmotto
Copy link
Owner

Cheers dude, I'll update the Readme unless you have time (see changelog at bottom) :)

@borislit
Copy link
Contributor Author

borislit commented Jan 9, 2016

@toddmotto pushed the dist files. I dont mind changing README, but ill have to do it a bit later :)
Lets see who gets there first :)

Glad to help :)

@borislit borislit changed the title Align with later NG 1.5 changes Align with latest NG 1.5 changes Jan 9, 2016
toddmotto added a commit that referenced this pull request Jan 9, 2016
Align with latest 1.5 changes
@toddmotto toddmotto merged commit fc61886 into toddmotto:master Jan 9, 2016
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.

2 participants