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

Improve the top section of config description #4387

Merged
merged 1 commit into from
Aug 28, 2018
Merged

Conversation

mmattel
Copy link
Contributor

@mmattel mmattel commented Aug 27, 2018

This is an improved description text of the top section of config.sample file.
It does not touch the samples from core but only the general description.

@mmattel mmattel requested a review from settermjd August 27, 2018 12:19
.. Do not edit this file; edit the source file in core
.. Do not edit the content of this file between _section_start and _sections_end
.. The content there will be loaded and replaced via script from the source file from link above
.. Any configuration changes done in this file will be overwritten on next update
Copy link
Contributor

Choose a reason for hiding this comment

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

s/on/on the/g

.. Do not edit the content of this file between _section_start and _sections_end
.. The content there will be loaded and replaced via script from the source file from link above
.. Any configuration changes done in this file will be overwritten on next update
.. You can of course change the common desription above this text which will be then part of the next update
Copy link
Contributor

Choose a reason for hiding this comment

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

s/be then/then be/g

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.

Just a few changes. Otherwise, it looks good, thanks.

@mmattel
Copy link
Contributor Author

mmattel commented Aug 27, 2018

Added the changes as requested


.. note:: ownCloud may write configurations into the ``config.php`` file.
These configurations may confict with identical keys already set in additional config files.
Be carefule when using this capability!
Copy link
Contributor

Choose a reason for hiding this comment

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

s/carefule/careful

@mmattel
Copy link
Contributor Author

mmattel commented Aug 28, 2018

Added the changes as requested

@settermjd settermjd merged commit 17ef30c into master Aug 28, 2018
@settermjd settermjd deleted the Improve_config.php branch August 28, 2018 10:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants