[sellnat77]

Description

• Attempts to use the current asset's exifInfo.description in the bound description (When it exists)
• This approach will set a default value for the description if we have it and differs from the previous approach of trying to set the value of the text area

Fixes #4073
Fixes #6244

How Has This Been Tested?

sorry for the buggy PR @waclaw66 Any other test paths you want me to exercise?

• Upload image -> Add description -> Add to shared album -> Open image as photo owner -> Open shared link as photo owner -> open shared link in incognito tab(anon user) -> Verify photo description is visible across all viewing methods
• Upload image AFTER testing above ^ -> Verify existing descriptions are maintained and web doesnt crash
• Add image without description to shared album -> Verify asset detail is correct
• Modify image description after upload -> Verify description is maintained in standard view + album views

Screenshots (if appropriate):

Left: As Photo Middle: As photo owner through shared link Right: As anon user in incognito

Tested the photo detail when people are present to exercise the code within the if statement

After editing description just to make sure the detail is updated properly

Checklist:

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation if applicable

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
immich	⬜️ Ignored (Inspect)	Visit Preview		Oct 18, 2023 2:26am

[jrasm91]

Did you test navigating to the next/previous assets, opening and closing the asset in each view?

[sellnat77]

Ohhhhh nice catch ty! next/prev doesnt show the description, working on it

[jrasm91]

Stale, closing. Feel free to re-open if you find a way to make this work correctly.

[jrasm91]

Oh you know what, I mis-read that you had actually already fixed my comment. I tested this and it does indeed allow for descriptions in shared links.