-
Notifications
You must be signed in to change notification settings - Fork 6.4k
Js update #9
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
base: main
Are you sure you want to change the base?
Js update #9
Conversation
just corrected a few typos and made some moderate modifications to wording
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.
Looks good!
page, you'll see a **Resources** section that lists one resource: `/articles`. | ||
The server has read the `db.json` file and found our `articles` key, turning it | ||
into a resource. Click `/articles` and you will be navigated to a new page, | ||
> **Note:** If you see an error saying something like `EADDRESSINUSE`, that |
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.
Should this note come after the paragraph below it instead?
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.
That does seem like a better place for it. Changed!
README.md
Outdated
`25.666 ms` is the amount of time it took to complete the request, in | ||
milliseconds. Back in Postman, we can see the response from our server in the | ||
lower panel and confirm we received what we expected. | ||
JSON Server sees that this is a GET request. |
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.
Suggest combining this line with the paragraph below
Co-authored-by: Liz Burton <[email protected]>
…rver-and-postman into js-update
No description provided.