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

Added changes again to support Authorization Code flow. #57

Merged
merged 1 commit into from
Nov 25, 2014

Conversation

raymondelferink
Copy link
Contributor

Your changes in 6947b1c on lines 210 and 309 break the support for Authorization Code flow.

@raymondelferink raymondelferink changed the title Your changes in https://github.com/andreareginato/oauth-ng/commit/6947b1c7088c34e833494fb14d0e972e0835b64c on lines 210 and 309 break the support for Authorization Code flow. Added changes again to support Authorization Code flow. Nov 21, 2014
andreareginato added a commit that referenced this pull request Nov 25, 2014
Added changes again to support Authorization Code flow.
@andreareginato andreareginato merged commit ddfc0ac into angularjs-oauth:master Nov 25, 2014
@andreareginato
Copy link
Collaborator

Thansk a lot @raymondelferink. Things should be ok now!

@andreareginato
Copy link
Collaborator

One thing. There were some tests failing. Please give it a check next time ;)

@raymondelferink
Copy link
Contributor Author

Hi Andrea,

Thanks and sorry, when I made the pull request last week all seemed fine.

Raymond
Op 25 nov. 2014 08:34 schreef "Andrea Reginato" [email protected]:

One thing. There were some tests failing. Please give it a check next time
;)


Reply to this email directly or view it on GitHub
#57 (comment)
.

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