Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

Splitting occ_command.rst into two files for core and apps #4381

Merged
merged 5 commits into from
Aug 28, 2018

Conversation

mmattel
Copy link
Contributor

@mmattel mmattel commented Aug 21, 2018

This PR splits the current occ_command.rst into two files:

occ_command.rst core related
and
occ_apps_command.rst apps related

occ_command.rst contains all commands for the core system.

occ_apps_command.rst contains now all commands for apps provided by ownCloud.
I went thru all repros and added all commands I have identified and described on best effort.

owncloud/core#32387 (Splitting config.sample.php into two files for core and app)

@settermjd please let me know or pls do by yourself how to add the new file into the doc structure to make it visible.

ToDo: Harmonize the layout to tables. Should be done in a seperate PR

FYI @PVince81 @DeepDiver1975 @patrickjahns @phil-davis

@settermjd
Copy link
Contributor

Thanks @mmattel

@mmattel mmattel force-pushed the occ_apps_command branch 3 times, most recently from b0940e5 to 1d3adcc Compare August 22, 2018 11:45
@mmattel
Copy link
Contributor Author

mmattel commented Aug 22, 2018

@settermjd info

I have added and improved in the meanwhile a lot of content in the new occ_apps_command.rst file.
If you watch via restview you will see how professional this looks now 😃
This is the way of UI changes I would like to see in occ_command.rst file too (to be made in seperate PR´s)

Copy link
Contributor

@settermjd settermjd left a comment

Choose a reason for hiding this comment

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

This all looks good to me. But it still needs to be linked into a table of contents before it can be approved.

@mmattel
Copy link
Contributor Author

mmattel commented Aug 24, 2018

The changes you have made for the occ_apps_command.rst description on top (before the commands are described), could you do those changes in occ_command.rst too so we have that in sync ?

Update: obsolete, I added some general text and synced the files

Copy link
Contributor

@settermjd settermjd left a comment

Choose a reason for hiding this comment

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

Thanks for all the hard work on this one.

@settermjd settermjd merged commit 47c1dbf into master Aug 28, 2018
@settermjd settermjd deleted the occ_apps_command branch August 28, 2018 11:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants