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
Node revamp: Routing #27796
Node revamp: Routing #27796
Conversation
Switched from single to double quotes for inter-lesson consistency.
Spans and IDs added as necessary.
The assignment section is very light, with just Express' own primer/mini-guide, and a reminder about docs. I was unable to find any "standout" resources on this topic that were both limited in scope to routing, and provided more beyond just repeating what the lesson and the Express docs say. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaoShizhong Another fantastic lesson 🔥
Concise and covers everything it needs to. Left a couple comments below
TheOdinProject/theodinproject#4507 makes level 4 headings clickable with IDs, just like level 3 headings, so the subsection headings are sufficient with spans with IDs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 🚀 🚀
Amend Youtube link example to better reflect more of the examples given in the subsection. Also fixed "route" that should be called "request".
Sneaky post-approval commit - just enhancing the query params section to be more explicit about the syntax, and a small nit fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm (2) 🚀🚀
Because
Draft new Express routing lesson as part of the Node revamp project.
This PR
Routing
lessonIssue
Closes #27605
Additional Information
Pull Request Requirements
location of change: brief description of change
format, e.g.Intro to HTML and CSS lesson: Fix link text
Because
section summarizes the reason for this PRThis PR
section has a bullet point list describing the changes in this PRIssue
section