fix(toh-5): Fixed issues in tutorial flow

[brandonroberts]

The HeroDetailComponent should be in the module declarations from the beginning.

Closes #2096, #2097, #2132

[brandonroberts]

@wardbell @chalin please review

[chalin]

Yep, the HeroDetailComponent is missing from way back when it was introduced in toh-3. Maybe toh-4 needs to be checked too?

[chalin]

Nit. Drop this blank line.

[chalin]

Minor tweaks suggested, along with the need for keeping a few lines for Dart. Other than that, it LGTM.

(Btw, the apparent problem with the tutorial flow was just a symptom; the root cause was because the starting state of the app.module.ts didn't match the end state of the previous chapter).

[brandonroberts]

I made the suggested fixes and checked the toh-3 and toh-4 modules. The HeroDetailComponent was correctly included in those tutorials.

[brandonroberts]

I also updated the app.module.ts for toh-6 for consistency

[chalin]

I think Jade requires indentation levels to be in multiple of two characters, otherwise it can cause strange problems. Hence

• Line 484 should be indented by 4 spaces (not 3)
• Line 485 should be indented by 6 spaces (not 5).

[brandonroberts]

Issues fixed

[chalin]

LGTM, if Travis is happy too.

[wardbell]

I hijacked your PR to also use InMemoryWebApiModule everywhere instead of the providers.