Skip to content

docs: add Notable Changes to Desktop App 7.1.0 release notes#179

Merged
LukasHirt merged 2 commits into
masterfrom
notable-changes-7.1
Jun 18, 2026
Merged

docs: add Notable Changes to Desktop App 7.1.0 release notes#179
LukasHirt merged 2 commits into
masterfrom
notable-changes-7.1

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

Follow-up to #178 (merged), addressing the review feedback from review 4526651338.

Adds a Notable Changes subsection to the Desktop App 7.1.0 release-notes entry, matching the
format of the existing 6.0.x entries. The original 7.1.0 entry had only the General paragraph and
GitHub changelog link; this surfaces the items users need to know before upgrading.

Notable Changes (verified against owncloud/client CHANGELOG.md@v7.1.0)

Breaking removals first, then raised platform minimums, then UX enhancements:

  • Remove support for ownCloud Server (OC10) — client now requires Infinite Scale (#12167)
  • Remove the command-line sync client (#12162)
  • Remove bandwidth throttling controls (#12335)
  • Remove the suffix VFS plugin (#12174)
  • Remove the capability to upload conflict files (#12369)
  • Remove the activity tab from account settings (#12307)
  • macOS 13 minimum (#12245); Qt 6.8 minimum (#12270)
  • New account setup wizard (#12189); group-policy / system-based configuration (#12367); redesigned account & folder management view (#12399)

Notes on the other review points

  • The "version 10 or 11" wording in the IMPORTANT note is intentionally kept as-is.
  • "security fixes" boilerplate in General left unchanged — consistent with the rest of the file.

Verification

Built locally with npm run antora-local (Node 22): exit 0, no log lines referencing
desktop_release_notes.adoc. Confirmed all 11 bullets and their PR links render under 7.1.0,
above 6.0.3, with admonitions intact.

🤖 Generated with Claude Code

Add a Notable Changes list to the 7.1.0 entry, matching the format of
the existing 6.0.x entries. Surfaces the breaking removals that affect
existing users (OC10 support, command-line sync client, bandwidth
throttling, suffix VFS plugin), the raised platform minimums (macOS 13,
Qt 6.8), and the main UX enhancements. Verified against the upstream
owncloud/client CHANGELOG.md at tag v7.1.0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>

@DeepDiver1975 DeepDiver1975 left a comment

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.

🤖 Automated review by Claude Code review agent.

Overview

This PR adds a Notable Changes subsection to the Desktop App 7.1.0 entry in modules/ROOT/pages/desktop_release_notes.adoc, directly addressing the gap flagged in the review of the now-merged #178 (the 7.1.0 entry previously had only the General paragraph and a GitHub changelog link). The change is +15/-0, single file, additive only.

I cross-checked all 11 entries against the upstream source of truth: owncloud/client CHANGELOG.md at tag v7.1.0 (release dated 2026-06-08). Every PR number and description matches the upstream changelog, including exact wording for the macOS/Qt minimums and the wizard/config/management enhancements. Accuracy is solid.

Code quality / style

  • AsciiDoc is well-formed: [discrete] + === Notable Changes heading and * Change: / * Enhancement: bullets mirror the existing 6.0.x sections exactly. Good consistency.
  • All 11 inline link macros (https://...pull/NNNNN[#NNNNN]) are syntactically valid and the link text matches the target PR number in each case.
  • Section placement is correct: under 7.1.0, above the 6.0.3 section, with the existing IMPORTANT/NOTE admonitions left intact above it.

Specific suggestions (all optional / non-blocking)

  1. Capitalization of "MacOS" (#12245 bullet). The bullet reads "MacOS 13 is now the minimum supported macOS version" — this faithfully copies the upstream changelog, but the file's own IMPORTANT admonition and the adjacent Qt bullet use the canonical "macOS". Consider normalizing to "macOS 13 ..." for in-page consistency. (Upstream uses "MacOS", so keeping it is defensible — editorial call.)
  2. Subset selection is reasonable but worth a sanity check. The upstream 7.1.0 changelog lists three additional Change items not surfaced here: Update sync exclusion list for Infinite Scale (#12093), Improve Windows VFS error messages (#12371), and Distinguish moves vs. renames in the activity view (#12494). Omitting bugfix-adjacent/minor changes from a curated "Notable Changes" list is the right editorial choice; flagging only so the omission is known to be deliberate. The OC10-removal bullet appropriately leads the list.
  3. Minor wording nuance (non-blocking). The upstream detail for Qt 6.8 (#12270) notes it "also raises the minimum macOS deployment target to 12," which sits slightly alongside the separate "macOS 13 minimum" bullet (#12245). Both bullets are individually accurate to upstream; no change needed, just noting the two macOS-version statements coexist.

Potential issues / risks

  • None blocking. No broken xrefs (all links are external absolute URLs to owncloud/client PRs, which resolve), no AsciiDoc parse risks, version string 7.1.0 and the implied release date (2026-06-08 upstream) are correct, and the change is purely additive so it cannot regress other sections.
  • Build verification was reported in the PR description (npm run antora-local, exit 0); the diff contains nothing that would plausibly break the Antora build.

Verdict

Accurate, well-formatted, and faithful to the upstream changelog. Recommend merge. The only nit worth considering before merge is the "MacOS" vs "macOS" casing for in-page consistency.

Use the canonical 'macOS' spelling to match the rest of the page
(the IMPORTANT note and the adjacent Qt bullet), per review feedback.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
@LukasHirt LukasHirt merged commit a4ad7d2 into master Jun 18, 2026
2 checks passed
@LukasHirt LukasHirt deleted the notable-changes-7.1 branch June 18, 2026 20:34
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.

2 participants