Skip to content

docs: show CLI updater before web updater in update.rst#14939

Open
skjnldsv wants to merge 1 commit into
masterfrom
fix/updater-cli-before-web
Open

docs: show CLI updater before web updater in update.rst#14939
skjnldsv wants to merge 1 commit into
masterfrom
fix/updater-cli-before-web

Conversation

@skjnldsv
Copy link
Copy Markdown
Member

☑️ Resolves

What changed and why

Most self-hosted Nextcloud admins have SSH access, and updater.phar is
the more reliable approach (no web server timeout risk, and the Troubleshooting
section already recommends it over the web-based updater when things go wrong).

The previous order put "Using the web based updater" first and
"Using the command line based updater" (updater.phar) second, so occ upgrade
appeared in the docs before updater.phar was introduced.

This PR swaps the section order so updater.phar is presented first:

  1. Using the command line based updater
  2. Batch mode for command line based updater
  3. Using the web based updater

Also removes a now-stale forward reference ("steps are basically the same
as for the web based updater") from the CLI section intro.

🖼️ Screenshots

No layout changes — section reorder only.

✅ Checklist

  • I have built the documentation locally and reviewed the output
  • Screenshots are included for visual changes (N/A — no visual changes)
  • I have not moved or renamed pages (or added a redirect if I did)
  • I have run codespell or similar and addressed any spelling issues

Most users with a self-hosted Nextcloud have SSH access and the CLI
updater (updater.phar) is the more reliable approach. Move the
"Using the command line based updater" and "Batch mode" sections
before "Using the web based updater" so the preferred method is
presented first.

Also remove the forward reference ("same as for the web based updater")
from the CLI section intro since the web section now follows it.

Fixes #7605

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

📖 Documentation Preview

🔍 Open preview →

📄 1 changed documentation page

Last updated: Wed, 20 May 2026 05:45:35 GMT

@skjnldsv skjnldsv requested review from kesselb and szaimen and removed request for kesselb May 20, 2026 05:53
@skjnldsv skjnldsv self-assigned this May 20, 2026
@skjnldsv skjnldsv requested review from kesselb and miaulalala May 20, 2026 05:53
@skjnldsv skjnldsv enabled auto-merge May 20, 2026 05:53
Copy link
Copy Markdown
Contributor

@miaulalala miaulalala left a comment

Choose a reason for hiding this comment

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

Move section

Using the command line based updater
------------------------------------

The command line based updater performs the same steps and checks as the
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This can move to the web updater instead

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't get it 🤔

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since the web updater is now after the command line updater, it would make sense to add this to the web updater - i. e. "The web updater performs the same steps..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

please reverse order "php occ upgrade" vs. "php updater/updater.phar"

2 participants