Skip to content

Conversation

@stewx
Copy link
Contributor

@stewx stewx commented Sep 14, 2025

Description

This forces an update of the person thumbnail in the asset viewer detail panel when you set a new featured photo.

Fixes #21967

How Has This Been Tested?

Manual testing

Screenshots (if appropriate)

stalepicfix.mp4

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

Claude Code generated the bug fix

@stewx stewx force-pushed the bugfix/stale-feat-photo branch from 5c4e2d7 to 434a125 Compare September 14, 2025 16:21
@stewx stewx changed the title Refresh photo after updating featured photo fix: Refresh photo after updating featured photo Sep 14, 2025
@stewx stewx marked this pull request as ready for review September 14, 2025 16:54
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

Haven't tested it but the code looks good to me.

@alextran1502 alextran1502 merged commit 26e0cb3 into immich-app:main Sep 17, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Featured photo for person does not automatically update in UI

3 participants