Skip to content
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

docs: Miss double quotation on the db.json #55450

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

iamxiaobing47
Copy link

The id on the db.json should be with "", otherwise it returns Not Found error Reference: https://www.npmjs.com/package/json-server

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

The tutorial-14-http demonstrates how to mock a json-server and provides a db.json file. However, the application cannot connect to http://localhost:3000/locations/{id}, because as mentioned in the doc https://www.npmjs.com/package/json-server, the id should be with double quotes

Issue Number: N/A

What is the new behavior?
The application can connect to http://localhost:3000/locations/{id} now

Does this PR introduce a breaking change?

  • Yes
  • [x ] No

Other information

The id on the db.json should be with "", otherwise it returns Not Found error
Reference: https://www.npmjs.com/package/json-server
@angular-robot angular-robot bot added the area: docs Related to the documentation label Apr 21, 2024
@ngbot ngbot bot added this to the Backlog milestone Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Related to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant