-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update migration guide #1610
Update migration guide #1610
Conversation
✅ Deploy Preview for expressjscom-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thanks for the ping! I hadn't seen this until now. I am working on a blog post for the v5 release now, and it would be awesome if we could land this before so I can link to it. I also ping'd @danieldelcore who did some codemod work, would be good if we can team up for this. |
@wesleytodd It would be great to add codemod, it would make the transition to the new version much easier. I can help with that. |
cc/: @expressjs/express-tc @expressjs/triagers |
1796b8b
to
5fd7a0d
Compare
LGTM. I added @UlisesGascon specifically as a reviewer, since this is based on his comment in. #1479 (comment). @UlisesGascon If you approve, please go ahead and merge this so it doesn't have to wait on me. |
We can always make more changes in the future. I will merge it now 👍 |
* add `res.query` and `res.redirect` * add `express.urlencoded` and `req.body` * add `res.clearCookie` and `res.status` * update introduction page * add `res.redirect('back')` and `res.location('back')` * bring changes from expressjs#1317 * add `brotli` support * adapt router matching syntax
Several undocumented changes have been added in version 5, as referenced in #1479 (comment):