-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
chore(server): remove unused endpoint/service/interface in asset-v1 #9086
Conversation
…ected. Used regex to get necessary portion for comparison
Deploying immich with
|
Latest commit: |
f91def8
|
Status: | ✅ Deploy successful! |
Preview URL: | https://7f4e714d.immich.pages.dev |
Branch Preview URL: | https://web-remove-unused-endpoint.immich.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the app currently still using those? If yes we may want to wait a release or two, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, we just need to merge #9085 first
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So if mobile still depends on it next release will be a breaking change, right? Why not wait a few releases then until we remove the endpoint?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mobile doesn't use this anymore, it was part of the other pr where I removed unused code.
Removed unused endpoint in
asset-v1