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

feat: Allow Tags To Be Applied In Bulk On Search, PersonID, and Memory-Viewer Pages #14368

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Menghini
Copy link

I have been adding a lot of tags to my photos on my Immich server to help organize photos. However, when I try to add tags based upon a PersonID or when using the search function, I am forced right now to add it to a temporary album and then open the album to then apply the tag. This PR would solve that issue.

@Menghini
Copy link
Author

Menghini commented Nov 27, 2024

This PR would implement Feature Request #14106.

Although #14213 added some great features, it still was missing the tag option in the menu.

@danieldietzler danieldietzler changed the title Allow Tags To Be Applied In Bulk On Search and PersonID Pages feat: Allow Tags To Be Applied In Bulk On Search and PersonID Pages Nov 27, 2024
@danieldietzler
Copy link
Member

Please fix the formatting errors

@Menghini
Copy link
Author

Please fix the formatting errors

Should be fixed now, I believe.

@Menghini Menghini changed the title feat: Allow Tags To Be Applied In Bulk On Search and PersonID Pages feat: Allow Tags To Be Applied In Bulk On Search, PersonID, Memory-Viewer Pages Nov 28, 2024
@Menghini
Copy link
Author

Now also includes the Memory-Viewer page as well.

@Menghini Menghini changed the title feat: Allow Tags To Be Applied In Bulk On Search, PersonID, Memory-Viewer Pages feat: Allow Tags To Be Applied In Bulk On Search, PersonID, and Memory-Viewer Pages Nov 28, 2024
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.

LGTM

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.

3 participants