docs(docs-infra): first-app tutorial - minor corrections #60738
Conversation
Updated housing.service.ts > getHousingLocationById() - Added search parameter to fetch URL - Returns single object instead of array
Added HELPFUL note calling attention to subtle change made in prior code (commit 7e05f8c).
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
angular-robot
bot
added
area: docs
|
Deployed adev-preview for 6dd2b26 to: https://ng-dev-previews-fw--pr-angular-angular-60738-adev-prev-rqa1rlh9.web.app Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt. |
|
Hi, Could you please fix the formatting (cf the linting issue on |
JeanMeche
added
the
action: cleanup
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The tutorial as written does not properly fetch data after updating Housing Service to fetch from a JSON server.
const data = await fetch(`${this.url}/${id}`);Issue Number: N/A
What is the new behavior?
The fetch method now properly queries the DB by the
idproperty specified in the route.const data = await fetch(`${this.url}?id=${id}`);Does this PR introduce a breaking change?
Other information