fix(occ): Add user:auth-tokens to command listing
#12331
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
These were implemented in nextcloud/server#40026 Signed-off-by: Josh <josh.t.richards@gmail.com>
joshtrichards
requested review from
nickvergessen,
ChristophWurst and
lhsazevedo
|
/backport to stable30 |
|
/backport to stable29 |
|
/backport to stable28 |
This was referenced Oct 29, 2024
8 tasks
solracsf
reviewed
Oct 30, 2024
| user:add-app-password adds a app password named "cli" (deprecated: alias for user:auth-tokens:add) | ||
| user:auth-tokens:add Add app password for the named account | ||
| user:auth-tokens:delete Deletes an authentication token | ||
| user:auth-tokens:list List authentication tokens of an user |
There was a problem hiding this comment.
Just to be rephrased similarly to options below
Suggested change
| user:auth-tokens:list List authentication tokens of an user | |
| user:auth-tokens:list List specified user authentication tokens |
There was a problem hiding this comment.
occ user:auth-tokens:list --help
Description:
List authentication tokens of an user
If this should be reworded, it should be reworded in server first and then simply be aligned here
There was a problem hiding this comment.
Yeah more polish would certainly be nice on the embedded command descriptions. I'd like to go through most of them one day, but that's a bigger job. I don't have the time or patience for it right now, particularly relative to more pressing priorities.
solracsf
approved these changes
Oct 30, 2024
This was referenced Oct 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These were implemented in nextcloud/server#40026 but never documented.
No usage examples, but at least the commands are now listed.
☑️ Resolves
🖼️ Screenshots