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): save original file name with extension #7679
chore(server): save original file name with extension #7679
Conversation
Deploying with Cloudflare Pages
|
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 this still WIP? I don't see changes to the web or mobile to not append the extension anymore. I also don't see a migration.
@@ -0,0 +1,20 @@ | |||
import { MigrationInterface, QueryRunner } from 'typeorm'; | |||
|
|||
export class AddExtensionToOriginalFileName1709763765506 implements MigrationInterface { |
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.
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.
It looks fine. Only potential concern is that it doesn't handle filenames with no extension, but I think we reject any files without extensions anyway.
Co-authored-by: Mert <101130780+mertalev@users.noreply.github.com>
Can you fix this? immich/web/src/lib/utils/asset-utils.ts Line 162 in 3da2b05
|
* chore(server): save original file name with extension * extract extension * update e2e test * update e2e test * download archive * fix download archive appending name * pr feedback * remove unused code * test * unit test * remove unused code * migration * noops * pr feedback * Update server/src/domain/download/download.service.ts Co-authored-by: Mert <101130780+mertalev@users.noreply.github.com> --------- Co-authored-by: Mert <101130780+mertalev@users.noreply.github.com>
No description provided.