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

docs: add immich-upload-optimizer to Community Projects list #15738

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

miguelangel-nubla
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 27, 2025
@alextran1502
Copy link
Contributor

I am unsure about adding this project because it is performing file modification to the original file, which is opposite to our design's philosophy of not touching the original files.

@aviv926
Copy link
Contributor

aviv926 commented Jan 28, 2025

I am unsure about adding this project because it is performing file modification to the original file, which is opposite to our design's philosophy of not touching the original files.

Maybe we can add a specific note about this project, ultimately the user needs to know that this is part of what this side project does (modifies the original file to save space) if they choose to implement it.

@miguelangel-nubla
Copy link
Contributor Author

I understood Community Projects are separate projects and not PR's to the original project precisely because they do not fit the main project design's philosophy.

@alextran1502 alextran1502 merged commit c2ba1cc into immich-app:main Jan 28, 2025
34 of 35 checks passed
@miguelangel-nubla miguelangel-nubla deleted the patch-1 branch January 28, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:skip documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants