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

Allowing people to get the authorization url instead of forcing the redirection #31

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

Conversation

totophe
Copy link

@totophe totophe commented Nov 8, 2013

This mechanism allow a better integration with frameworks allowing the developer handling the redirection rather than forcing it.

@jabes
Copy link

jabes commented Sep 10, 2014

I could use this on my current project, any word on when this will be implemented?

@totophe
Copy link
Author

totophe commented Sep 10, 2014

I've just rewritten the part were it generate the url, moved it into a separate public function and "voilà" :-)

It should have been merged a long time ago, but it isn't :( You can use my branch, but it hasn't been updated in a while. I can probably do a pull from the galen repository if you want.

@odino
Copy link

odino commented Sep 10, 2014

ping @galen

@eschultz
Copy link

Sadly @galen passed away on 12/1/2013 due to non-hodgkins lymphoma. 😢

@odino
Copy link

odino commented Aug 13, 2017

RIP 😢

@totophe
Copy link
Author

totophe commented Aug 18, 2017

Sad news :'( RIP

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