Skip to content

Added text to explain what granting access to applications via the authentication process #1185

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

Merged
merged 1 commit into from
May 9, 2019

Conversation

megoth
Copy link
Contributor

@megoth megoth commented May 8, 2019

This would accompany the work being done on #1142, which would link to this resource to give users a bit more understanding on what they're about to do.

I've written this with non-technical users in mind.

This would accompany the work being done on #1142, which would link to this resource to give users a bit more understanding on what they're about to do.

I've written this with non-technical users in mind.
@megoth megoth requested a review from jaxoncreed May 8, 2019 19:18
Copy link
Contributor

@jaxoncreed jaxoncreed left a comment

Choose a reason for hiding this comment

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

Looks good to me :shipit:

Is the intention to link to this document on github? It's not an html file, so we're not serving it up from the server.

@jaxoncreed jaxoncreed merged commit 51cd35c into master May 9, 2019
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