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

Add Amazon provider #157

Closed

Conversation

@martincostello
Copy link
Member

@martincostello martincostello commented May 6, 2017

This PR adds a provider to use for Login with Amazon.

Resolves #156.

Add provider for Amazon for #156.
Add myself to the contributors list in the README file.
@martincostello
Copy link
Member Author

@martincostello martincostello commented May 14, 2017

@PinpointTownes What kind of timescale is there for this being reviewed, merged and published to NuGet?

@kevinchalet
Copy link
Member

@kevinchalet kevinchalet commented May 14, 2017

@PinpointTownes What kind of timescale is there for this being reviewed, merged and published to NuGet?

Sorry for not getting back to you sooner. Reviewing and merging all the PRs of this repo is on my todo list, but I'm currently overbooked with my professional activities. I'll try to dedicate an hour or two next week.

Thanks.

@martincostello
Copy link
Member Author

@martincostello martincostello commented May 14, 2017

👍

@kevinchalet
Copy link
Member

@kevinchalet kevinchalet commented May 21, 2017

Merged, thanks for your contribution! 👏

12cc086

@martincostello martincostello deleted the martincostello:Add-Amazon-Provider branch May 21, 2017
@kevinchalet kevinchalet modified the milestones: 1.0.0, 1.0.0-rc2 May 21, 2017
martincostello added a commit to martincostello/alexa-london-travel-site that referenced this pull request Jun 21, 2018
Use AspNet.Security.OAuth.Amazon for Amazon authentication instead of the original code that was contributed by aspnet-contrib/AspNet.Security.OAuth.Providers#157.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants