immich icon indicating copy to clipboard operation
immich copied to clipboard

feat(web): Add Partner Sharing Avatars to Timeline and Info Cards

Open x24git opened this issue 1 year ago • 7 comments
trafficstars

This Feature adds user avatars to the timeline for both partner sharing, and shared albums.

Timeline

The avatar uses the user's avatar image (if one is set) or the server's information is used to make an avatar using the user's assigned color and first letter of their name. (In the screenshot below, the partner's icon is an Orange 'T'). Implements a simple store for user data to efficiently pull user data for the timelines. image

New Settings

In order to control this feature there is a new setting added in the sharing config (changed from partner sharing to generic sharing). image

Asset View

Additionally, there is improvements to the asset view to make it clear the source of the image (either shared, when in an album, or from partner sharing in the main timeline) image

Is the companion feature for the same capability set on the mobile app #12089

x24git avatar Sep 06 '24 22:09 x24git

Implements Feature Request https://github.com/immich-app/immich/discussions/7608

x24git avatar Sep 06 '24 22:09 x24git

You might want to rebase - the diff is looking pretty gnarly

mertalev avatar Oct 16 '24 03:10 mertalev

Cleaned up commit history and rebased on main. Thanks @mertalev

x24git avatar Oct 16 '24 03:10 x24git

Resolved all merge conflicts and test cases. Ready to merge. (Apologies for opening and reopening)

x24git avatar Oct 16 '24 05:10 x24git

Maybe it should be placed under Partner Sharing? image

aviv926 avatar Oct 17 '24 20:10 aviv926

Maybe it should be placed under Partner Sharing?

Agreed, updated the PR description with the new placement. Swapped out the Partner Sharing title to a generic Sharing configuration, and then moved partners into an accordion box. Allow for future sharing specific settings to be added in the future. image

x24git avatar Oct 17 '24 21:10 x24git

Maybe it should be placed under Partner Sharing?

Agreed, updated the PR description with the new placement. Swapped out the Partner Sharing title to a generic Sharing configuration, and then moved partners into an accordion box. Allow for future sharing specific settings to be added in the future. image

Looks good I like that, but it's not my job to decide so let's see what the opinion of the core development team is.

aviv926 avatar Oct 18 '24 07:10 aviv926

Closing due to stale. We can look into this at a later time

alextran1502 avatar May 08 '25 13:05 alextran1502