Skip to content

Update migration guide - #1610

Merged
UlisesGascon merged 8 commits into
expressjs:gh-pagesfrom
bjohansebas:migration-v5
Oct 6, 2024
Merged

Update migration guide#1610
UlisesGascon merged 8 commits into
expressjs:gh-pagesfrom
bjohansebas:migration-v5

Conversation

@bjohansebas

@bjohansebasbjohansebas commented Sep 12, 2024•
edited
Loading

Copy link
Copy Markdown
Member

Several undocumented changes have been added in version 5, as referenced in #1479 (comment):

@netlify

netlifyBot commented Sep 12, 2024•
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for expressjscom-preview ready!

NameLink
🔨 Latest commit5fd7a0d
🔍 Latest deploy loghttps://app.netlify.com/sites/expressjscom-preview/deploys/66ea03eeaad7a1000834c378
😎 Deploy Previewhttps://deploy-preview-1610--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 site configuration.

@bjohansebasbjohansebas added the 5.x Docs for 5.x version label Sep 12, 2024
@bjohansebas
bjohansebas marked this pull request as ready for review September 13, 2024 01:08
@bjohansebas
bjohansebas requested a review from a teamSeptember 13, 2024 01:08
@wesleytodd

wesleytodd commented Sep 13, 2024

Copy link
Copy Markdown
Member

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.

@bjohansebas

bjohansebas commented Sep 13, 2024

Copy link
Copy Markdown
MemberAuthor

@wesleytodd It would be great to add codemod, it would make the transition to the new version much easier. I can help with that.

@bjohansebasbjohansebas mentioned this pull request Sep 14, 2024
Closed
bjohansebas
bjohansebas commented Sep 14, 2024
View reviewed changes
Comment threaden/guide/migrating-5.md Outdated
@bjohansebas
bjohansebas requested a review from a teamSeptember 16, 2024 21:58
@bjohansebas

bjohansebas commented Sep 16, 2024

Copy link
Copy Markdown
MemberAuthor

cc/: @expressjs/express-tc @expressjs/triagers

@crandmck

crandmck commented Sep 29, 2024

Copy link
Copy Markdown
Member

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.

@bjohansebasbjohansebas mentioned this pull request Oct 1, 2024
Closed
@UlisesGascon

UlisesGascon commented Oct 6, 2024

Copy link
Copy Markdown
Member

We can always make more changes in the future. I will merge it now 👍

@UlisesGascon
UlisesGascon merged commit 5bbc22a into expressjs:gh-pagesOct 6, 2024
@bjohansebas
bjohansebas deleted the migration-v5 branch October 6, 2024 16:08
chrisdel101 pushed a commit to chrisdel101/expressjs.com that referenced this pull request Nov 3, 2024
@bjohansebasChris Del
Update migration guide (expressjs#1610)
c1d9a4f
* 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
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blakeembreyblakeembreyblakeembrey left review comments
@crandmckcrandmckcrandmck approved these changes
@UlisesGasconUlisesGasconUlisesGascon approved these changes

Assignees

No one assigned

Labels

5.xDocs for 5.x version

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bjohansebas@wesleytodd@crandmck@UlisesGascon@blakeembrey