Skip to content

compatibility with angular 1.6#146

Open
rondoe wants to merge 4 commits into
angularjs-oauth:masterfrom
rondoe:angular-1.6
Open

compatibility with angular 1.6#146
rondoe wants to merge 4 commits into
angularjs-oauth:masterfrom
rondoe:angular-1.6

Conversation

@rondoe

@rondoe rondoe commented Dec 15, 2016

Copy link
Copy Markdown

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
Copy Markdown

+1

@Luwangel

Luwangel commented Mar 3, 2017

Copy link
Copy Markdown

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

rondoe commented Mar 7, 2017

Copy link
Copy Markdown
Author

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

@mathieujobin

Copy link
Copy Markdown

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