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

Doc: use api.admin #312

Merged
merged 2 commits into from
Jul 26, 2023
Merged

Doc: use api.admin #312

merged 2 commits into from
Jul 26, 2023

Conversation

gctucker
Copy link
Contributor

Update the Getting Started documentation page to use the new api.admin module and delete init_user_groups as it's not used any more.

Update the Getting Started section to refer to the new api.admin
module with the setup_admin_user thin wrapper rather than the
create_admin_user script which has already been removed.

Signed-off-by: Guillaume Tucker <[email protected]>
Remove the init_user_groups script now that we have api.admin to
create the initial admin user group.

Signed-off-by: Guillaume Tucker <[email protected]>
Copy link
Collaborator

@JenySadadia JenySadadia left a comment

Choose a reason for hiding this comment

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

LGTM.
Let's wait until it gets reflected on the staging doc.

@gctucker
Copy link
Contributor Author

OK it's there now: https://static.staging.kernelci.org/docs/api/getting-started/#create-an-admin-user-account

For some reason it got stuck, it should have updated itself last night automatically. Investigating...

@JenySadadia
Copy link
Collaborator

Looks good on staging: https://static.staging.kernelci.org/docs/api/getting-started/#create-an-admin-user-account

@JenySadadia JenySadadia added this pull request to the merge queue Jul 26, 2023
Merged via the queue into kernelci:main with commit de34011 Jul 26, 2023
@gctucker gctucker deleted the doc-api-admin branch July 26, 2023 11:14
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