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

V2 endpoints #27

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

V2 endpoints #27

wants to merge 15 commits into from

Conversation

c-robison
Copy link

Changed endpoints to use v2. Added properties to allow implicit flow support.

@c-robison c-robison closed this Mar 16, 2017
@glangston83 glangston83 deleted the v2_endpoints branch March 16, 2017 16:07
@HNeukermans
Copy link
Owner

Hi @c-robison,

Seems like a valueable contribution. Can, I ask you why did you close the PR?

@c-robison
Copy link
Author

Hi @HNeukermans,

I had meant to merge the branch back to the fork master is all. Most of the stuff I changed, like the v2 endpoint and sessionStorage vs. localStorage may not be useful for you. I could reopen and let you make that decision if you like.

@HNeukermans
Copy link
Owner

Yeah cool,
if you can, just reopen it. I will take a look, at what parts I can use...

@c-robison c-robison restored the v2_endpoints branch March 17, 2017 22:54
@c-robison c-robison reopened this Mar 17, 2017
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