Skip to content

Conversation

@dolfje
Copy link
Contributor

@dolfje dolfje commented Feb 9, 2026

…featured asset

Description

With the person, there is a featured asset shown. But after deleting that asset delivering the thumbnail, the same asset is still shown as thumbnail. This can be triggered by 2 path. Either by deleting the asset or by deleting the person from the asset. In both paths, the asset should not be part of the person, so definitely not be used as thumbnail.

So as solution, a randomly new asset is chosen and afterwards the thumbnail is generated in a job.

How Has This Been Tested?

steps to reproduce:

  • goto the explore page
  • open one of the people
  • open the asset that is used as featured photo
  • on the people side panel, click the edit Button
  • click the delete button of the person
  • press the back button (to go back to timeline of the person)

expected result:
The featured photo of the person is adapted

actual result:
The featured photo is changed => medium test provided

steps to reproduce:

  • goto the explore page
  • open one of the people
  • open the asset that is used as featured photo
  • on asset viewer, delete that photo
  • press the back button (to go back to timeline of the person)

expected result:
The featured photo of the person is adapted

actual result:
The featured photo is changed => medium test provided

Checklist:

  • I have performed a self-review of my own code
  • [n/a] 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.

To create testcode, but no original code of the LLM is left...

@immich-push-o-matic
Copy link

immich-push-o-matic bot commented Feb 9, 2026

Label error. Requires exactly 1 of: changelog:.*. Found: 🗄️server. A maintainer will add the required label.

Nikos Verschore added 2 commits February 10, 2026 00:30
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.

1 participant