-
Notifications
You must be signed in to change notification settings - Fork 56
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
Adds new documentation #803
Adds new documentation #803
Conversation
Corrects admin documentation Edits images on the user website documentation page
docs/admin/servers/auth.md
Outdated
|Application ID|CLIENT_ID| _xx_ | | ||
|Application Secret|CLIENT_SECRET| _xx_ | | ||
|Callback URL|(to be directly entered in Gitlab OAuth registration)|| | ||
|Forward-auth secret|OAUTH_SECRET|_random-secret-string_ (password for forward-auth, can be changed to your preferred string) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line length
docs/admin/guides/remove_user.md
Outdated
|
||
- Go to the _docker_ directory | ||
|
||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/remove_user.md
Outdated
|
||
- Remove these three lines from the `conf.server` file | ||
|
||
```txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/remove_user.md
Outdated
|
||
- Run the command for these changes to take effect: | ||
|
||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/add_user.md
Outdated
|
||
**3. Add username and password:** | ||
2. Add three lines to the `conf.server` file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ordered list item prefix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The formatting is off when this page is rendered as html
Fixes codeclimate issues on existing docs
docs/admin/guides/remove_user.md
Outdated
|
||
- Go to the _docker_ directory | ||
|
||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/remove_user.md
Outdated
|
||
- Remove these three lines from the `conf.server` file | ||
|
||
```txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/remove_user.md
Outdated
|
||
- Run the command for these changes to take effect: | ||
|
||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/add_user.md
Outdated
--restart always \ | ||
mltooling/ml-workspace-minimal:0.13.2 | ||
``` | ||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/add_user.md
Outdated
|
||
The following code adds basic authorization for the new user. | ||
```txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/servers/auth.md
Outdated
Select option to generate client secret and also selection option | ||
for trusted application. | ||
|
||
2. **Determine Website Hostname** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
docs/admin/servers/auth.md
Outdated
It's recommended to use a self-hosted GitLab instance, which you will use in | ||
other parts of the DTaaS application. | ||
|
||
3. **Determine Callback and Logout URLs** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
docs/admin/servers/auth.md
Outdated
The logout URL is to help users logout of traefik forward-auth. The logout | ||
URL should not be entered into Gitlab OAuth application setup. | ||
|
||
4. **Create OAuth Application** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
docs/admin/servers/auth.md
Outdated
|
||
![Creation of Server OAuth Application](server-oauth.png) | ||
|
||
5. **Copy Application Credentials** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
docs/admin/servers/auth.md
Outdated
![Server OAuth Application Credentials](server-oauth2.png) | ||
|
||
<!-- markdownlint-disable MD013 --> | ||
1. **Checklist: Required Information from OAuth Application:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
Updates libms docs
|GitLab Variable Name|Variable Name in .env of docker compose file |Default Value| | ||
|---|---|---| | ||
<!-- markdownlint-disable MD013 --> | ||
| GitLab Variable Name | Variable Name in .env of docker compose file | Default Value | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line length
docs/admin/client/auth.md
Outdated
@@ -35,17 +29,21 @@ Here are the steps to get started: | |||
homepage of the DTaaS application. | |||
- The logout URL is where users will be directed after logging out. | |||
|
|||
**4. OAuth Application Creation:** | |||
**4. OAuth Application Creation:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing spaces
docs/admin/guides/add_user.md
Outdated
|
||
**4. Add 'route' for new user:** | ||
```bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code block style
docs/admin/guides/remove_user.md
Outdated
|
||
Please do the following: | ||
|
||
**1. Remove an existing user with the [DTaaS CLI](../cli.md)** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Emphasis used instead of a header
Code Climate has analyzed commit a551e6d and detected 2 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
89c9125
into
INTO-CPS-Association:feature/distributed-demo
Builds on PR #786