Skip to content

docs(ocis): add 8.2 updates — clean-corrupt-public-shares CLI - #63

Merged
mmattel merged 1 commit into
mainfrom
docs/ocis-8.2-updates
Sep 14, 2026
Merged

mmattel merged 1 commit into
mainfrom
docs/ocis-8.2-updates

Conversation

@DeepDiver1975

@DeepDiver1975 DeepDiver1975 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Port of owncloud/docs-ocis#1285 ("docs: updates required for 8.2", by @mmattel) into the monorepo.

That PR never merged upstream, so ocis 8.2 shipped without this content and no version folder here carries it.

Important

Base branch is docs/ocis-8.1-updates (#62), not main — a deliberate chain, because both ports edit commands.adoc in content/ocis/8.2 and 8.3. Basing on #62 keeps this diff to its own 7 files instead of showing a conflict. Upstream's own PR body says the same thing: "it is important to merge this PR AFTER the 8.1 work has finished".

Merge order: #61 (merged) → #62 → this. Retarget to main once #62 lands.

Where the changes land

Substantive content → content/ocis/8.2 and 8.3:

Change File
new Clean Corrupt Public Shares page (+33) admin/pages/maintenance/commands/clean-corrupt-public-shares.adoc
its index entry (inserted before Cleanup Orphaned Grants) admin/pages/maintenance/commands/commands.adoc
the command added to the COMMANDS: help listing, column re-aligned admin/partials/maintenance/ocis-shares-cli.adoc

Version bookkeeping → content/ocis/8.2 only:

Change File
:description: → 8.1.0 → 8.2.0 delta, and the delta bullet list admin/pages/maintenance/commands/changed-cli.adoc

Corrected while porting

Upstream wrote the delta as "between … 8.0.0 and 8.1.0" — a leftover from when the branch's base was still the 8.1 dev line, which contradicts the PR's own "updates required for 8.2" title. Corrected to 8.1.0 → 8.2.0, the delta that content/ocis/8.2 actually describes.

This is also why the two ports no longer collide: #62 sets the 8.1 folder's delta page, this one sets the 8.2 folder's, and neither touches the other's.

Verification

  • npm run antora → exit 0. Error/warning count is 41, identical to docs(ocis): add 8.1 updates — vault mode, blobstore CLI, 8.0→8.1 upgrade guide #62's — i.e. this PR adds none. Zero mentions of clean-corrupt, corrupt-public or shares-cli in the log. The 28 target of xref not found: next@ocis:… errors are pre-existing in content/main/modules/ROOT/pages/ocis_release_notes.adoc, untouched here.
  • npm test → 17/17 pass.
  • clean-corrupt-public-shares.html renders in both public/ocis/8.2/ and public/ocis/8.3/.
  • Per-version delta pages read as designed: 8.1 → 8.0.0 and 8.1.0, 8.2 → 8.1.0 and 8.2.0, 8.3 → unchanged.

Refs: owncloud/docs-ocis#1285.

Addresses the upstream issue this content was written for, now migrated into this repo: #85 (owncloud/docs-ocis#1275).
Refs #44

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mmattel
mmattel force-pushed the docs/ocis-8.1-updates branch from e454d54 to dc5eb4a Compare September 14, 2026 12:04
Base automatically changed from docs/ocis-8.1-updates to main September 14, 2026 13:23
Port of owncloud/docs-ocis#1285, which never merged upstream -- so ocis 8.2
shipped without this content and no version folder here carries it.

Applied to content/ocis/8.2 and 8.3 (substantive content, per the README
backport rule):

  * new "Clean Corrupt Public Shares" command page
  * its index entry in commands.adoc
  * the command added to the ocis-shares-cli help listing

Applied to content/ocis/8.2 only (version bookkeeping):

  * changed-cli.adoc: 8.1.0 -> 8.2.0 delta

The upstream PR wrote that delta as "between 8.0.0 and 8.1.0" -- a leftover
from when its base branch was the 8.1 dev line. Corrected to 8.1.0 -> 8.2.0,
which is the delta the content/ocis/8.2 folder describes. The 8.1 folder's
own delta page is set by the companion 8.1 port, so the two no longer collide.

Refs: owncloud/docs-ocis#1285
Refs: owncloud/docs-ocis#1275
Co-Authored-By: Martin Mattel <github@diemattels.at>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@mmattel
mmattel force-pushed the docs/ocis-8.2-updates branch from 18c7fb4 to 5bfb225 Compare September 14, 2026 13:24
@mmattel
mmattel marked this pull request as ready for review September 14, 2026 13:26
@mmattel
mmattel requested a review from a team as a code owner September 14, 2026 13:26
@mmattel

mmattel commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Lets merge this and to the checks later

@mmattel
mmattel merged commit 3be853f into main Sep 14, 2026
2 of 3 checks passed
@mmattel
mmattel deleted the docs/ocis-8.2-updates branch September 14, 2026 13:26
oc-tmueller added a commit that referenced this pull request Sep 16, 2026
Verification probe for the change in this PR, not content. A `lint` check-run
must now appear on this new head sha; before the change, a push produced none
(see #116, #110, #63 in the PR description).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Signed-off-by: Thomas Müller <323649642+oc-tmueller@users.noreply.github.com>
oc-tmueller added a commit that referenced this pull request Sep 16, 2026
A check-run belongs to the head sha it ran against, so without `synchronize` every push left the new head with no `lint` run — perpetually pending for a required context. Prerequisite for requiring `lint` in owncloud/admin. Verified on this PR: the pushed head got its own lint run, unlike #116/#110/#63.
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.

4 participants