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

Example for custom apps directories does not work correctly #739

Closed
wants to merge 1 commit into from
Closed

Example for custom apps directories does not work correctly #739

wants to merge 1 commit into from

Conversation

julijane
Copy link
Contributor

@julijane julijane commented Apr 4, 2018

This configuration example does not work because apps shipped with Nextcloud can
not be updated.

There is a current example. Nextcloud 13.0.1 ships with files_pdfviewer 1.2.0, but an update to 1.2.1 is available. However when using a configuration as shown in the example (and as similarly used in nextcloud/docker) updating does not work. The update is downloaded and installed into the second "writable" app directory, however Nextcloud will still load the old version as shipped and continue to complain about an update being available.

Changing the order so that the "writable" folder comes first fixed this. Updates are put there and Nextcloud uses them. This is arguably a bug in the server but I am not sure about that.

See also nextcloud/docker#308

This configuration example does not work because apps shipped with Nextcloud can
not be updated. Changing the app directory order fixed that.
@julijane julijane changed the title Example for custom_apps directories does not work correctly Example for custom apps directories does not work correctly Apr 4, 2018
@tflidd
Copy link
Contributor

tflidd commented Apr 5, 2018

From the same doc:

To ensure that the default /apps/ folder only contains apps shipped with Nextcloud, follow this example to setup an /apps2/ folder which will be used to store all other apps.

So I'm asking myself if this is not a wanted behavior that you can only update non-core apps. Why do you want a write protection on your apps folder in the first place?

related: nextcloud/server#8929

@skjnldsv
Copy link
Member

@julijane this was an issue in our code, this has been fixed now :)

@skjnldsv skjnldsv closed this Jun 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants