Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: change image response to preview instead of download #235

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xyTom
Copy link
Collaborator

@xyTom xyTom commented Feb 24, 2025

No description provided.

@xyTom xyTom requested a review from Copilot February 24, 2025 07:38
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

functions/file/[id].js:104

  • Ensure that the new behavior of setting 'Content-Disposition' to 'inline' is covered by tests.
    // Create a new response with modified headers for image preview

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants