Version 7.3.0

[ildyria]

Summary by CodeRabbit

• New Features
  • Enterprise LDAP/Active Directory user authentication (Supporter Edition)
  • Photo details drawer now opens by default
• Bug Fixes
  • Hotfix: Docker image now includes php-ldap dependency
  • Various stability improvements and migration fixes
• Documentation
  • Troubleshooting note for LDAP/php-ldap errors after updating
  • Installation requirements updated to list ldap PHP extension
  • Expanded contributor guidelines and album rights notes
• Other
  • Release announcement updated to v7.3.1

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds releases v7.3.1 (hotfix) and v7.3.0 (feature) to release notes and roadmap, updates announcement to v7.3.1, increments total releases to 112, adds PHP ldap to installation requirements, and inserts a troubleshooting entry about php-ldap / facade runtime error (duplicated in the FAQ).

Changes

Cohort / File(s)	Summary
Release notes docs/releases.md	Inserted v7.3.1 (hotfix) and v7.3.0 (feature) sections; expanded v7.3.0 "Most notable changes"; removed two lines from the v7.2.1 subsection.
Site pages / UI src/components/widgets/Announcement.astro, src/pages/roadmap.astro	Updated announcement text/link to v7.3.1; added v7.3.1 and v7.3.0 entries to the roadmap and updated dashboard stats (Latest Version -> 7.3.1, Total Releases -> 112).
Troubleshooting / FAQs docs/faq_troubleshooting.md	Added troubleshooting subsection about "PHP Fatal error: ... A facade root has not been set" explaining php-ldap is required; the entry was added in two locations (duplicated).
Installation requirements docs/installation.md	Added ldap (php-ldap) to the PHP extensions server requirements list (new since 7.3).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I hopped through changelogs, bright and spry,
Two releases baked beneath the sky,
I nudged in ldap so things run neat,
A tiny fix made builds complete,
I twitch my nose and tap my feet.

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 2