Closed RushilJalal closed 1 week ago
Thanks for this PR @RushilJalal
The current link to the http module only is intentional. We want people to be aware of the whole http module doc, and at this point in the curriculum, people should have no issues searching it for a specific method. Having a link to the method then also linking the module doc as a whole is redundant in my honest opinion.
@MaoShizhong I get your point. But the docs page doesn't have a search option. I know one could just Ctrl + F
it tho.
If you want the users to be aware of the http
module the link should be
https://nodejs.org/api/http.html#http
not
https://nodejs.org/api/http.html
What do you think?
@MaoShizhong I get your point. But the docs page doesn't have a search option. I know one could just
Ctrl + F
it tho. If you want the users to be aware of thehttp
module the link should be https://nodejs.org/api/http.html#http not https://nodejs.org/api/http.html What do you think?
The original link is fine as it's all one page, and the original link is the top of the page with the full table of contents. Everything there is part of the http
module.
I would be up for rewording that assignment step's first sentence to something like this to better reflect its intent:
Check out the docs for Node's http module. In particular, check out
http.createServer
.
Lesson Link : https://www.theodinproject.com/lessons/nodejs-getting-started
Because
Previous link provided was for
http
module, nothttp.createServer
This PR
http.createServer
docsPull 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