Skip to content

Conversation

@maynkxx
Copy link

@maynkxx maynkxx commented Nov 11, 2025

Summary

This PR updates the Express 4 migration guide to include information about the send.mime change introduced in [email protected].

Details

The send module removed its default MIME type fallback in version 0.18.0. Applications that relied on implicit MIME type detection must now configure MIME behavior manually.

Changes Made

  • Updated en/guide/migrating-4.md to document the send.mime fallback removal.
  • Added a reference link to the official send release notes.

Related Issue

Fixes #1868

Checklist

  • Documentation updated
  • Verified formatting and Markdown consistency
  • References provided for upstream module change

@maynkxx maynkxx requested a review from a team as a code owner November 11, 2025 06:07
@netlify
Copy link

netlify bot commented Nov 11, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit d61eefd
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6912d5a67135d60008057f94
😎 Deploy Preview https://deploy-preview-2114--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

🚦 Lighthouse Results (Mobile & Desktop)

URL Device Perf A11y Best Practices
/ mobile 🟠 87 🟢 100 🟢 96
/en/blog/posts.html mobile 🟢 97 🟢 96 🟢 96
/en/5x/api.html mobile 🟢 97 🟢 95 🟢 96
/ desktop 🟢 100 🟢 100 🟢 96
/en/blog/posts.html desktop 🟢 100 🟢 96 🟢 92
/en/5x/api.html desktop 🟢 100 🟢 95 🟢 96

@maynkxx
Copy link
Author

maynkxx commented Nov 11, 2025

Hi , this PR adds documentation for the send.mime removal in the migration guide as discussed in issue #1868.
All checks have passed successfully.

Please review when you have a moment. Thank you!

@efekrskl
Copy link
Contributor

Thanks for the PR! It looks like you removed 501 lines, and added none with your last commit :)

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.

Add the changes in send to the migration guide

2 participants