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

Unify optional settings documentation for REMOTE_AUTH*/SOCIAL_AUTH* #149

Closed
jathanism opened this issue Mar 15, 2021 · 3 comments · Fixed by #191
Closed

Unify optional settings documentation for REMOTE_AUTH*/SOCIAL_AUTH* #149

jathanism opened this issue Mar 15, 2021 · 3 comments · Fixed by #191
Assignees
Labels
type: documentation Improvements or additions to documentation type: housekeeping Changes to the application which do not directly impact the end user
Milestone

Comments

@jathanism
Copy link
Contributor

Proposed Changes

Currently, the documentation for setting up default groups/permissions for remotely-authenticated users is either duplicated or spread out across the documentation for Remote User Authentication and SSO Authentication.

We should fix that and, move them to be documented to a single location and reference those docs from each authentication guide.

Justification

Currently it's not clear on whether these settings would also work with LDAP or other forms of authentication, but they absolutely should. The documentation makes it seem as if these are distinct and there are different ways. Additionally the LDAP documentation does not cover or link to these configuration settings at all.

@jathanism jathanism added type: documentation Improvements or additions to documentation status: help wanted Extra attention is needed status: current labels Mar 15, 2021
@jathanism jathanism added this to the v1.0.0 milestone Mar 15, 2021
@dgjustice
Copy link
Contributor

@jathanism I'll take a look at this one.

@jathanism
Copy link
Contributor Author

@dgjustice You're hired!

@dgjustice
Copy link
Contributor

I guess this issue is a duplicate of #18

@jathanism jathanism added this to In progress in Release v1.0.0 Mar 24, 2021
@jathanism jathanism added type: housekeeping Changes to the application which do not directly impact the end user and removed status: help wanted Extra attention is needed labels Mar 24, 2021
Release v1.0.0 automation moved this from In progress to Done Mar 30, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: documentation Improvements or additions to documentation type: housekeeping Changes to the application which do not directly impact the end user
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants