Skip to content

[10.4] Document the enable and disable options to occ app:list#2144

Merged
phil-davis merged 6 commits intomasterfrom
2141-document-additions-to-occ-app_list
Jan 10, 2020
Merged

[10.4] Document the enable and disable options to occ app:list#2144
phil-davis merged 6 commits intomasterfrom
2141-document-additions-to-occ-app_list

Conversation

@settermjd
Copy link
Contributor

This fixes #2141 by documenting the enable and disable options, including the fact that version numbers are displayed in the output, when those options are used. This commit also:

  • cleans up the section's formatting
  • cleans up the command's documentation; and
  • adds in missing options (for completeness)

It relates to owncloud/core#36520.

This fixes #2141 by documenting the above options, along with the note
about the fact that version numbers are displayed in the output, when
those options are used. This commit also cleans up the sections
formatting, cleans up the command's documentation, and adds in missing
options for completeness.
phil-davis
phil-davis previously approved these changes Dec 9, 2019
Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

LGTM
Only merge when the core PR is merged (so far nobody has officially reviewed it for inclusion in core 10.4)

@settermjd
Copy link
Contributor Author

LGTM
Only merge when the core PR is merged (so far nobody has officially reviewed it for inclusion in core 10.4)

Understood.

@settermjd settermjd added the 10.4 label Dec 9, 2019
@settermjd settermjd changed the title Document the enable and disable options to occ app:list [10.4] Document the enable and disable options to occ app:list Dec 9, 2019
@settermjd settermjd added the waiting for release requires a release of ownCloud or app before it can be merged label Dec 9, 2019
After reviewing owncloud/core#36520, I felt that
the wording in this PR could be simpler still, hence this change.
phil-davis
phil-davis previously approved these changes Dec 12, 2019
@phil-davis
Copy link
Contributor

The core PR owncloud/core#36520 has been merged and should be in 10.4 release.

@settermjd settermjd requested a review from phil-davis January 10, 2020 13:04
@phil-davis phil-davis merged commit 7a78a65 into master Jan 10, 2020
@delete-merged-branch delete-merged-branch bot deleted the 2141-document-additions-to-occ-app_list branch January 10, 2020 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for release requires a release of ownCloud or app before it can be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update of occ command reference necessary

2 participants