Repository navigation
Add note about not placing files inside /var/www/html - #1168
myzhang1029 wants to merge 1 commit into
Conversation
Existing files inside /var/www/html get deleted by the upgrade script. This update documents that. (#1165) Signed-off-by: Zhang Maiyun <myzhang1029@hotmail.com>
| nextcloud | ||
| ``` | ||
|
|
||
| Please note that anything inside `/var/www/html` will be removed by the installation script, except for `config`, `data`, `custom_apps`, `themes` and `version.php`. So make sure you do not leave other important files inside. |
There was a problem hiding this comment.
Yeah, I agree. That would probably be better.
There was a problem hiding this comment.
| Please note that anything inside `/var/www/html` will be removed by the installation script, except for `config`, `data`, `custom_apps`, `themes` and `version.php`. So make sure you do not leave other important files inside. | |
| Please note that anything inside `/var/www/html` will be removed by the installation script, except for those listed in [upgrade.exclude](https://github.com/nextcloud/docker/blob/master/upgrade.exclude). So make sure you do not leave other important files inside. |
|
I'd love to help get this one over the line - in addition to the issue mentioned above there's also #1528. This looks like a straight-forward solution compared to #1548. With regard to the proposed addition, would a slight tweak to the wording help make clear that this issue is relevant for both installation, and upgrades? E.g.:
I'm not sure what the status of this PR is but I'm happy to submit a new one if required. |
|
I think you need to start from scratch with this PR. It's too outdated. |
|
Fixed in #1862 |
Existing files inside /var/www/html get deleted by the upgrade script. This update documents that. (#1165)