Skip to content

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Sep 19, 2025

Summary by CodeRabbit

  • Documentation
    • Added release notes for v6.9.2 (Sep 19, 2025) and v6.9.1 (Aug 31, 2025), covering: Import from Server returning to the Web UI, new album subtitle display options, tag migration fix, guidance and fixes for subfolder deployments, an option to disable server imports for security, S3 migration support for photos, and docs on adding OAuth providers.
  • Style
    • Updated site announcement to promote Lychee v6.9.2.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 19, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds two patch release notes (v6.9.2 — Sep 19, 2025 — and v6.9.1 — Aug 31, 2025) to release documentation and updates the site announcement banner to reference v6.9.2. Edits are textual; one commented link text remains inconsistent.

Changes

Cohort / File(s) Summary
Documentation: Release notes
docs/releases.md
Added v6.9.2 (Sep 19, 2025) and v6.9.1 (Aug 31, 2025) entries. Notes include subfolder handling fix, guidance on subfolder insecurity, OAuth provider docs (#3673), S3 migration command (#3684), import-from-server env toggle (#3694), and subfolder duplicate-path fix (#3696). v6.9.1 includes Web UI import-from-server (#3647), album subtitle display options (#3662), and tag migration fix (#3659).
UI: Announcement banner
src/components/widgets/Announcement.astro
Updated visible banner anchor and href to reference v6.9.2; commented href updated to v6.9.2 while the commented display text still mentions v6.9.1.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit drums a gentle beat,
“v6.9.2 — how neat!”
Docs refreshed, the banner's bright,
Import hops back into sight.
Carrots raised to patches two — onward we leap! 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "version 6.9.2" directly and succinctly identifies the primary change—the release bump to v6.9.2 reflected in the changelog and announcement widget—so it is relevant and concise. It clearly communicates the main purpose of the PR to a reviewer scanning history. The only minor issue is stylistic (lowercase and no "Release" or "v" prefix), which does not undermine clarity.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bf09475 and e2774a3.

📒 Files selected for processing (1)
  • docs/releases.md (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
src/components/widgets/Announcement.astro (1)

12-14: Update all version references consistently.

There's an inconsistency in the version updates. While the visible announcement correctly references v6.9.2, the text in the comment (Line 18) still shows "Lychee v6.9.1 is now available!" which is inconsistent with the updated href.

Apply this diff to maintain consistency throughout the comment:

-    class="text-slate-200 hover:underline dark:text-slate-200 font-medium"><span class="text-red-500 font-bold">CVSS 7.5 in Lychee [6.6.6 to 6.6.9], update as soon as possible!</span> Lychee v6.9.1 is now available! »</a
+    class="text-slate-200 hover:underline dark:text-slate-200 font-medium"><span class="text-red-500 font-bold">CVSS 7.5 in Lychee [6.6.6 to 6.6.9], update as soon as possible!</span> Lychee v6.9.2 is now available! »</a
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d62cb9 and bf09475.

📒 Files selected for processing (2)
  • docs/releases.md (1 hunks)
  • src/components/widgets/Announcement.astro (1 hunks)
🔇 Additional comments (2)
docs/releases.md (2)

33-61: LGTM! Well-structured release notes.

The v6.9.2 release notes are comprehensive and well-organized. The security warning about subfolder deployments is properly highlighted, and the documentation on new features (OAuth providers, S3 migration, import controls) is clear and informative. The examples provided for subfolder configuration are helpful for users.

The release notes appropriately reference security improvements and new functionality that aligns with the Lychee photo management system.


63-98: LGTM! Excellent documentation of the import feature restoration.

The v6.9.1 release notes clearly document the return of the "Import from server" feature with proper context about its history (available in v4, removed in v5, now restored in v6). The notes about job queue recommendations for large imports and the technical relationship to the CLI lychee:sync command are valuable for users.

Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com>
@ildyria ildyria merged commit af9ad4d into master Sep 19, 2025
1 of 2 checks passed
@ildyria ildyria deleted the version-6.9.2 branch September 19, 2025 07:23
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