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

feat(admin): document the --showonly flag of the app update command #11399

Merged

Conversation

SimJoSt
Copy link
Contributor

@SimJoSt SimJoSt commented Dec 14, 2023

πŸ–ΌοΈ Screenshots

Screenshot 2023-12-14 at 01 05 12

Signed-off-by: Joda Stâßer <git@simjo.st>
@SimJoSt
Copy link
Contributor Author

SimJoSt commented Dec 14, 2023

Source of the information of the existence of this flag: https://help.nextcloud.com/t/check-for-app-updates/83403/4

Copy link
Contributor

@tflidd tflidd left a comment

Choose a reason for hiding this comment

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

Code was added a long time ago: nextcloud/server#11053
and it is a follow up on this #1653

Co-authored-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux merged commit 1abc721 into master Jan 18, 2024
11 checks passed
@susnux susnux deleted the feat(admin)-document-the-`--showonly`-flag-of-the-update-command branch January 18, 2024 13:14
Copy link
Contributor Author

@SimJoSt SimJoSt left a comment

Choose a reason for hiding this comment

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

It sounds better after the change. I was trying to keep the phrasing similar to the name of the flag πŸ˜„

@tflidd
Copy link
Contributor

tflidd commented Jan 31, 2024

/backport to stable28

@tflidd
Copy link
Contributor

tflidd commented Jan 31, 2024

/backport to stable27

@tflidd
Copy link
Contributor

tflidd commented Jan 31, 2024

/backport to stable26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants