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

compatibility with angular 1.6 #146

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rondoe
Copy link

@rondoe rondoe commented Dec 15, 2016

fixed the changes in angular 1.6. for the promise api. success replaced to .then callback.
bower json is original. tested with angular 1.6.

@rondoe rondoe changed the title upgrade to angular 1.6 compatibility with angular 1.6 Dec 15, 2016
@raftheunis87
Copy link

+1

@Luwangel
Copy link

Luwangel commented Mar 3, 2017

You change the indentation of so many files, it's almost impossible to see the diffs. But at the end of the day I think this is almost the same change as the #145.

@rondoe
Copy link
Author

rondoe commented Mar 7, 2017

yep. i'm sorry for that, it was my atom config. great to have that issue fixed.

@mathieujobin
Copy link

unfortunately too much white space changes to review.
I can add ?w=1 to the URL, but still.

there are unrelated changes that should not be part of this PR

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.

4 participants