New Lesson: Basic REST API#29090
New Lesson: Basic REST API#29090Lofty-Brambles wants to merge 24 commits intoTheOdinProject:mainfrom
Conversation
Maddily
left a comment
There was a problem hiding this comment.
Great work overall. I suggested a few minor changes to improve the flow and clarity.
HenriqueHeron
left a comment
There was a problem hiding this comment.
Went through the article and it's looking great. I've made a few corrections and grammar improvement suggestions.
Asartea
left a comment
There was a problem hiding this comment.
General comment: this lesson uses the term URI 4 times, and URL 9 times. Is there (for the content of this lesson) a meaningful difference expressed with that? Otherwise it should be probably use one of those consistently.
Excluding that and a few minor nits I think overall it looks great
Co-authored-by: Asartea <76259120+Asartea@users.noreply.github.com> Co-authored-by: Mayada <115709272+Maddily@users.noreply.github.com> Co-authored-by: Henrique Heron <44820991+autotelico@users.noreply.github.com>
Co-authored-by: Mayada <115709272+Maddily@users.noreply.github.com>
Maddily
left a comment
There was a problem hiding this comment.
Made a few other small tweaks for readability.
Co-authored-by: Mayada <115709272+Maddily@users.noreply.github.com>
|
Apologies for the delay. Completely forgot about it. |
mao-sz
left a comment
There was a problem hiding this comment.
Nit: For consistency with the revamped lessons, instead of "ExpressJS", it should be referred to simply as "Express".
Occurs in text on lines 98, 109 and 110.
|
@MaoShizhong @01zulfi - if it's all right, this should be good to merge? |
|
@Lofty-Brambles Yea, everything looks good. I'll merge all the PRs related to the milestone at a later date, most likely together |
|
|
||
| - [What is CRUD?](https://www.codecademy.com/article/what-is-crud) | ||
| - [What are the four main HTTP methods, and how are they used?](#crud-and-http-methods) | ||
| - [Which HTTP method corresponds to each action in CRUD (Create, Read, Update, Delete)?](https://stackoverflow.blog/2020/03/02/best-practices-for-rest-api-design/#h-use-nouns-instead-of-verbs-in-endpoint-paths) |
There was a problem hiding this comment.
Given the KC question being asked, linking to the appropriate lesson section feels better than linking to an external site.
| - [Which HTTP method corresponds to each action in CRUD (Create, Read, Update, Delete)?](https://stackoverflow.blog/2020/03/02/best-practices-for-rest-api-design/#h-use-nouns-instead-of-verbs-in-endpoint-paths) | |
| - [Which HTTP method corresponds to each action in CRUD (Create, Read, Update, Delete)?](#crud-and-http-methods) |
|
I also wonder if this lesson is an appropriate point to drop a quick link to Postman, perhaps at the end of the RESTful APIs section, e.g.
Then it can be re-linked as a reminder in the JWT project (TBD). I was originally going to use it in the JWT lesson but since we're streamlining the "tutorial", I can't find as good a place for an introductory link there and this lesson seems very appropriate IMO. The JWT lesson feels like a better place to reference Postman as if it's already been introduced. Thoughts @01zulfi @Lofty-Brambles ? |
|
Just realised that Postman is already first introduced in the Routes lesson, but I still think a quick re-mention of Postman here might be valuable. Thoughts? |
| ### Additional resources | ||
|
|
||
| This section contains helpful links to related content. It isn't required, so consider it supplemental. | ||
|
|
||
| - [HTTP Request/Response Basics](http://justahelp.blogspot.com/2013/09/http-requestresponse-basics.html) from Pralay Roy goes into further detail about their lifecycle and anatomy with best practices. |
There was a problem hiding this comment.
@Lofty-Brambles not sure you're aware but one of the big things the team is going through with the whole curriculum is re-evaluating additional resources and the purpose of the AR section.
Rereading this lesson, I'm wondering how you feel about this AR you've added. Do you feel it's important enough to the lesson to need mentioning on top of everything else covered? If so, is it something that can be integrated into the lesson itself in some way as opposed to as an AR? Or is there a good reason it should not be dropped but must still be separate from the lesson proper?
Because
New lesson on Basic REST APIs. Part of Milestone
#2of the Node Revamp.This PR
#2.Issue
Closes #28832
Additional Information
The Markdown Preview tool does not render the nested list under assignments and tables properly. The style for those was copied from other, currently active lessons.
Pull Request Requirements
location of change: brief description of changeformat, e.g.Intro to HTML and CSS lesson: Fix link textBecausesection summarizes the reason for this PRThis PRsection has a bullet point list describing the changes in this PRIssuesection