Skip to content

Show a person's age with their name - #681

Open
alangrainger wants to merge 1 commit into
immichFrame:mainfrom
alangrainger:feature/show-people-age
Open

Show a person's age with their name#681
alangrainger wants to merge 1 commit into
immichFrame:mainfrom
alangrainger:feature/show-people-age

Conversation

@alangrainger

@alangrainger alangrainger commented Jul 22, 2026

Copy link
Copy Markdown

Hi, love ImmichFrame, I use it everywhere!

I have some frames at my parents' house, and they wanted to see the age of the grandkids on each photo.

This PR adds the age of the person in brackets after their name, e.g. John (7). If there is no birthday set for that person in Immich, then just the name is shown.

The feature is gated on a new config option ShowPeopleAge (default false), and of course on the existing ShowPeopleDesc.

Summary by CodeRabbit

  • New Features

    • Added an optional setting to display each person’s age alongside their name in photo metadata.
    • Age display requires person descriptions and available birthday information.
    • The setting is disabled by default and is available through configuration files and environment variables.
  • Documentation

    • Updated configuration guides and examples with the new age display option.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ca3f6642-8ea0-4262-9197-1f04f1d18452

📥 Commits

Reviewing files that changed from the base of the PR and between 319affe and 9398746.

📒 Files selected for processing (19)
  • ImmichFrame.Core/Interfaces/IClientSettings.cs
  • ImmichFrame.WebApi.Tests/Resources/TestV1.json
  • ImmichFrame.WebApi.Tests/Resources/TestV2.json
  • ImmichFrame.WebApi.Tests/Resources/TestV2.yml
  • ImmichFrame.WebApi/Helpers/Config/ServerSettingsV1.cs
  • ImmichFrame.WebApi/Models/ClientSettingsDto.cs
  • ImmichFrame.WebApi/Models/ServerSettings.cs
  • Install_Web.md
  • docker/Settings.example.json
  • docker/Settings.example.yml
  • docker/example.env
  • docs/docs/getting-started/configuration.md
  • docs/docs/getting-started/configurationV1.md
  • immichFrame.Web/src/lib/components/elements/asset-component.svelte
  • immichFrame.Web/src/lib/components/elements/asset-info.svelte
  • immichFrame.Web/src/lib/components/elements/asset.svelte
  • immichFrame.Web/src/lib/components/home-page/home-page.svelte
  • immichFrame.Web/src/lib/immichFrameApi.ts
  • openApi/swagger.json

📝 Walkthrough

Walkthrough

This PR adds the ShowPeopleAge setting across server configuration, API contracts, frontend component props, person metadata rendering, configuration examples, documentation, and test resources.

Changes

ShowPeopleAge configuration and rendering

Layer / File(s) Summary
Configuration and API contract
ImmichFrame.Core/Interfaces/IClientSettings.cs, ImmichFrame.WebApi/Models/..., ImmichFrame.WebApi/Helpers/Config/..., immichFrame.Web/src/lib/immichFrameApi.ts, openApi/swagger.json
Adds ShowPeopleAge to server settings, adapters, client DTOs, TypeScript types, and the OpenAPI schema.
Frontend age rendering
immichFrame.Web/src/lib/components/elements/..., immichFrame.Web/src/lib/components/home-page/home-page.svelte
Forwards the setting through asset components and appends calculated ages to person names when dates are available.
Configuration examples and validation data
docker/..., Install_Web.md, docs/docs/getting-started/..., ImmichFrame.WebApi.Tests/Resources/...
Updates example configurations, installation documentation, configuration references, and versioned test resources.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested labels: maintenance

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main feature: showing a person’s age alongside their name.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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.

1 participant