Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unique filenames #148

Closed
wants to merge 8 commits into from
Closed

Unique filenames #148

wants to merge 8 commits into from

Conversation

tymmej
Copy link
Contributor

@tymmej tymmej commented Aug 29, 2023

Contains #146 as otherwise that option isn't useful.

tymmej added 2 commits August 29, 2023 18:06
otherwise there was empty folder with pre-normalized paths
@mandarons mandarons enabled auto-merge (rebase) August 29, 2023 17:26
@mandarons
Copy link
Owner

Does this PR make #146 and #145 obsolete?

@tymmej
Copy link
Contributor Author

tymmej commented Sep 21, 2023

This PR contains them.

I split them to smaller chunks to reduce scope and allow smaller merges.

@mandarons mandarons disabled auto-merge October 5, 2023 14:20
@mandarons
Copy link
Owner

mandarons commented Oct 5, 2023

@tymmej When all_albums is true, it will create nested folders for each album. That will create unique paths for photos that are present with same name in multiple albums.

@tymmej Why do we need this PR?

@tymmej
Copy link
Contributor Author

tymmej commented Oct 5, 2023

I'm using downloaded photos with https://github.com/photoview/photoview

Photoview has bug (or maybe it's intentional?) that photos in database are stored only by filename - not full path. So it doesn't work with multiple files with same filename, but in different folders.
Photoview displays folders as albums in web view, but if same photo is added to multiple albums in iCloud then it's visible only in one album in web view.
Adding this option allows to show photo in all albums.

@mandarons
Copy link
Owner

@tymmej: Okay. Yeah, I remember this use case. This is a one-off use case and doesn't matter for broader user base of this tool. I am not planning to spend any time in maintaining one-off use cases at this point. However, I can get this feature in, given that it is enabled by a flag in config and how important it is to you.

Going forward, would you like to be the maintainer of this feature?

@mandarons mandarons added this to the 1.15.0 milestone Oct 5, 2023
@tymmej
Copy link
Contributor Author

tymmej commented Oct 5, 2023

What "maintainer" does mean in this context?

Can this pull request wait? Immich made some progress with read-only libraries (https://documentation.immich.app/docs/features/libraries). It is not ideal at the moment - it doesn't create albums from folders, but maybe it will be added (immich-app/immich#4279). Also it seems that Immich has larger user base so I'm planning to migrate to it.

@mandarons
Copy link
Owner

@tymmej Maintainer for a feature is typically a go to person to get help from, should any issue arise in using the feature (i.e. helping with resolving bugs related to the feature).

Yeah, this PR can certainly wait. I only heard good things about immich. It seems to be trending to the top in FOSS photos management and being very actively developed. I certainly recommend migrating to immich.

@mandarons mandarons modified the milestones: 1.15.0, next-gen Oct 5, 2023
@mandarons mandarons added the awaiting-info Awaiting more information to proceed label Oct 5, 2023
@tymmej tymmej closed this Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-info Awaiting more information to proceed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants