Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(server-comm): copyedits #3373

Merged
merged 1 commit into from
Mar 14, 2017

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Mar 14, 2017

  • Minor prose copyedits.
  • Renamed HeroService.addHero() to create(), so as to match the ToH sample.

- Minor prose copyedits.
- Renamed `HeroService.addHero()` to `create()`, so as to match the ToH
sample.
Copy link
Contributor

@wardbell wardbell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. Do note that we have plans to redo this chapter. It should have more information on HTTP, not less, than the ToH!

@@ -21,32 +21,32 @@ import { Hero } from './hero';
@Injectable()
export class HeroService {
// #docregion endpoint
private heroesUrl = 'app/heroes'; // URL to web API
private heroesUrl = 'api/heroes'; // URL to web API
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we fixed that long ago but apparently not.

@wardbell wardbell merged commit 2fbca28 into angular:master Mar 14, 2017
@wardbell wardbell deleted the chalin-server-comm-0310 branch March 14, 2017 17:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants