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

Rs oauth article #32

Merged
merged 10 commits into from May 27, 2020
Merged

Rs oauth article #32

merged 10 commits into from May 27, 2020

Conversation

richardesigns
Copy link
Contributor

I have written an article about OAuth for the DID.app blog. I would be grateful if you could have a quick read and suggest any changes you think are necessary.

I have tried to write something for our technical audience that provides a fairly high-level introduction into OAuth as a concept. I have not tried to get into the detail of the specification.

See what you think anyway. My sources are all at the bottom of the article in 'further reading'

Copy link
Contributor Author

@richardesigns richardesigns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heya, thank you for your help with this. I've responded to each of your comments by adding inline reviews showing the diff. If you're happy with the way I've re-written these sections please do merge the changes in or let me know so I can merge them into this pull request.

@richardesigns richardesigns merged commit e8186da into master May 27, 2020
@richardesigns richardesigns deleted the rs-oauth-article branch May 27, 2020 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants