search

linuxserver / docker-nextcloud

GNU General Public License v3.0
678 stars 128 forks source link

Remove updatenotification #353

Closed nemchik closed 10 months ago

nemchik commented 10 months ago

All this app does is place a notification in the admin status page telling the user to click a button to run the web updater. We don't want this. Users should instead update the container image.

I tested the logic on my own instance. No issues. It runs completely silent if the apps don't exist.

Ref: https://github.com/nextcloud/all-in-one/blob/1771a72c0e5a37d43356c4d3e6c657dc5d4a0990/Containers/nextcloud/entrypoint.sh#L186-L192 https://github.com/nextcloud/all-in-one/blob/1771a72c0e5a37d43356c4d3e6c657dc5d4a0990/Containers/nextcloud/entrypoint.sh#L286-L287

Also added to https://github.com/linuxserver/docker-nextcloud/pull/352

LinuxServer-CI commented 10 months ago
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/nextcloud/27.0.2-pkg-59d697de-dev-ea2517e26d08f0e92909a2e0d8a34811fd2ab287-pr-353/index.html https://ci-tests.linuxserver.io/lspipepr/nextcloud/27.0.2-pkg-59d697de-dev-ea2517e26d08f0e92909a2e0d8a34811fd2ab287-pr-353/shellcheck-result.xml Tag Passed
amd64-27.0.2-pkg-59d697de-dev-ea2517e26d08f0e92909a2e0d8a34811fd2ab287-pr-353 ✅
arm64v8-27.0.2-pkg-59d697de-dev-ea2517e26d08f0e92909a2e0d8a34811fd2ab287-pr-353 ✅
LinuxServer-CI commented 10 months ago
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/nextcloud/27.0.2-pkg-59d697de-dev-fd7d9d1f0ad68dbc704422f18dfb49beee903240-pr-353/index.html https://ci-tests.linuxserver.io/lspipepr/nextcloud/27.0.2-pkg-59d697de-dev-fd7d9d1f0ad68dbc704422f18dfb49beee903240-pr-353/shellcheck-result.xml Tag Passed
amd64-27.0.2-pkg-59d697de-dev-fd7d9d1f0ad68dbc704422f18dfb49beee903240-pr-353 ✅
arm64v8-27.0.2-pkg-59d697de-dev-fd7d9d1f0ad68dbc704422f18dfb49beee903240-pr-353 ✅