Skip to content

refactor(server): immich file responses - #5641

Merged
alextran1502 merged 5 commits into
mainfrom
refactor/immich-file-response
Dec 12, 2023
Merged

alextran1502 merged 5 commits into
mainfrom
refactor/immich-file-response

Conversation

@jrasm91

@jrasm91 jrasm91 commented Dec 11, 2023

Copy link
Copy Markdown
Member

Move all res.sendFile logic to an interceptor that transforms ImmichFileResponse classes into res.sendFile calls.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 11, 2023 •

Copy link
Copy Markdown

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8170589
Status: ✅  Deploy successful!
Preview URL: https://e56f36e3.immich.pages.dev
Branch Preview URL: https://refactor-immich-file-respons.immich.pages.dev

View logs

@danieldietzler danieldietzler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat. Looks really good to me!

Comment thread server/src/immich/api-v1/asset/asset.controller.ts
@jrasm91
jrasm91 requested a review from mertalev December 11, 2023 22:38
@alextran1502
alextran1502 merged commit cbca698 into main Dec 12, 2023
@alextran1502
alextran1502 deleted the refactor/immich-file-response branch December 12, 2023 14:58
jonhnet pushed a commit to jonhnet/immich that referenced this pull request Dec 15, 2023
* refactor(server): immich file response

* chore: open api

* chore: tests

* chore: fix logger import

---------

Co-authored-by: Alex Tran <alex.tran1502@gmail.com>

This branch was previously deployed

1 inactive deployment
Preview — 81705898 Deployed Dec 12, 2023 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants