Address ID review items

[realModusOperandi]

Addressing comments in #27.

Questions: 1) Are async and await different features? That’s how I read it, so I updated the subject/verb agreement to be in line with my assumption. 2) What needs to be compatible with the features in, “To be compatible with these features,…”? I would like to update this to, “For ___ to be compatible with these features,…”.

1. They are the same feature, usually written as "async/await" so I updated to use that terminology and keep it as one noun.
2. The result of the HttpClient.get() call is what needs to be made compatible by invoking toPromise() on it, so I revised to say that.

Question: Do my updates look correct here? Is the lifecycle hook the “specific function” or the “component” mentioned in the sentence? I updated it so that it reads that the lifecycle hook is the function.

Yes these are correct.

Question: What are div and *ngFor (in terms of providing a noun after each)? The “the ___ {noun}” structure can make sentences more clear for users.

div is an element. *ngFor is a directive:

Now use the Angular ngFor directive in the template to display each item in the heroes list.

I've updated to use those terms in the format you requested.

[Charlotte-Holt]

@evelinec @realModusOperandi I made just a few small suggestions. Let me know if you have any questions. Thanks!

[Charlotte-Holt]

Change
"A good service performs only one function and it performs this function well."
to
"A good service performs only one function, and it performs this function well."

[Charlotte-Holt]

Check to make sure there isn't an extra space between "the" and "[hotspot=atInjectable]@Injectable"

[Charlotte-Holt]

Change
"The file imports the [hotspot=importHttpClient]HttpClient class and
[hotspot=importInjectable]Injectable decorator."
to
"The file imports the [hotspot=importHttpClient]HttpClient class and the [hotspot=importInjectable]Injectable decorator."

[Charlotte-Holt]

Change
"For the result of the [hotspot=httpInstanceAndAwaitFeatureAndHttpGetAndToPromise]HttpClient.get() method to be compatible with this feature, it must be converted to a Promise invoking its [hotspot=httpInstanceAndAwaitFeatureAndHttpGetAndToPromise]toPromise() method."
to
"For the result of the [hotspot=httpInstanceAndAwaitFeatureAndHttpGetAndToPromise]HttpClient.get() method to be compatible with this feature, it must be converted to a Promise by invoking its [hotspot=httpInstanceAndAwaitFeatureAndHttpGetAndToPromise]toPromise() method."

[Charlotte-Holt]

Change
"This component responds to the [hotspot=onInitInterface]OnInit event via the [hotspot=ngOnInitMethod]ngOnInit method which fetches and populates the component's template with data when it is initialized for display."
to
"This component responds to the [hotspot=onInitInterface]OnInit event via the [hotspot=ngOnInitMethod]ngOnInit method, which fetches and populates the component's template with data when it is initialized for display."

[realModusOperandi]

Ok, made the requested changes (and reformatted a line I missed.)

[Charlotte-Holt]

@realModusOperandi Looks good. Thanks, Liam. Approved from an ID perspective.

[evelinec]

Looks good. Will refresh the staging site to proceed with the final testing.