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

Corrections and additions to the authentication documentation section, tokens deleting...etc. #832

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

techcobweb
Copy link
Contributor

@techcobweb techcobweb commented Sep 26, 2024

Signed-off-by: Mike Cobbett [email protected]

Why ?

  • I noticed the explanation of how to set your galasactl up with the GALASA_TOKEN was wrong, so that needed correcting.
  • We needed to explain the command-line usage of users get --id me
  • We needed to explain the command-line usage of auth tokens get --user xxxx

@techcobweb
Copy link
Contributor Author

techcobweb commented Sep 26, 2024

@jt-nti I suspect i shouldn't be trying to merge into main ? Targeting next instead.

@techcobweb techcobweb changed the base branch from main to next September 26, 2024 09:28
Copy link

@aashir21 aashir21 left a comment

Choose a reason for hiding this comment

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

👍🏻

@jt-nti jt-nti merged commit 190bc53 into next Sep 26, 2024
19 checks passed
@jt-nti jt-nti deleted the mcobbett-auth-updates branch September 26, 2024 14:25
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.

3 participants