Skip to content

Conversation

ZackHoang
Copy link
Contributor

Because

Change req.body data access by accessing it through matchedData() rather than accessing it directly. Also add a note about why this is done and how req.query is not mutable in the current version of Express

This PR

  • Use matchedData() to access req.body
  • Add a note about req.query being unable to mutate

Issue

Closes #30190

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: NodeJS Involves the NodeJS course label Oct 6, 2025
@ZackHoang ZackHoang requested a review from mao-sz October 6, 2025 21:23
@ZackHoang
Copy link
Contributor Author

@mao-sz Sorry that you had to fix my entire PR lol 🤣

Copy link
Contributor

@mao-sz mao-sz left a comment

Choose a reason for hiding this comment

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

Nothing to apologise for

@mao-sz mao-sz merged commit fc7fc10 into TheOdinProject:main Oct 6, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: NodeJS Involves the NodeJS course
Projects
None yet
2 participants