You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Angular Router tutorial Milestone 3 missing information #22599
Confusing or missing steps in Milestone 3 of the Router section: https://angular.io/guide/router#add-heroes-functionality. The steps as given ask you to make a new directory, create an empty file, copy the contents of one TypeScript file (app.component.ts) into another (hero-list.component.ts), and then edit a template.
The issue is that it never asks you to copy over a specific template, and the TypeScript doesn't contain a template string. The possible html templates to copy over do not quite match the edits requested...or rather, two different ones do. The app.component.html has an h1 tag, but no h2 or hero detail. The heroes.component.html does have an h2 and hero detail, but no h1.
This then raises the question of whether the app.component.ts is what you're actually supposed to be copying from, or the heroes.component.ts, as neither html file has the entirety of code expecting to be modified, so it becomes unclear which typescript file is actually intended to be copied.
And then the list of files you are expected to have at the completion of this set of instructions contains only typescript. (hero-detail.component.ts, hero-list.component.ts, hero.service.ts, heroes.module.ts)
Expected behavior
The desired behavior would be to explain exactly which files need to be created or copied over, double-checking which file or project should be used to copy information over, and clarity about what should be copied where.
What is the motivation / use case for changing the behavior?
Clarity in training and tutelage of new angular developers.
Delete the selector (routed components don't need them).
Is this true, or does this mean to say "routed modules don't need them"? If it does intend to say routed components, I would request further clarification (or a link to further clarification) as to how a normal component isn't a "routed" component even if it is being loaded via router (or clarity in the Tour of Heroes tutorial that once you add routing you can remove selector from all components)
Activity
mmatlock-shr commentedon Mar 7, 2018
Additionally, I have a clarification question:
Is this true, or does this mean to say "routed modules don't need them"? If it does intend to say routed components, I would request further clarification (or a link to further clarification) as to how a normal component isn't a "routed" component even if it is being loaded via router (or clarity in the Tour of Heroes tutorial that once you add routing you can remove selector from all components)
jenniferfell commentedon May 15, 2018
docsarea=routing
brandonroberts commentedon Oct 8, 2018
Fixed via #20023
angular-automatic-lock-bot commentedon Sep 14, 2019
This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.
Read more about our automatic conversation locking policy.
This action has been performed automatically by a bot.