Skip to content

Conversation

@devversion
Copy link
Member

  • Updates the outdated schematics guide to mention the new available component schematics.

@devversion devversion added docs This issue is related to documentation pr: merge safe labels Oct 9, 2018
@devversion devversion requested a review from jelbourn as a code owner October 9, 2018 19:43
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 9, 2018
@devversion devversion changed the title docs: list all available schematics in the schematics guide docs: show all available schematics in the schematics guide Oct 9, 2018
* Updates the outdated `schematics` guide to mention the new available component schematics.
@devversion devversion force-pushed the chore/lupdate-schematics-md branch from 1994ff3 to 7a2521d Compare October 9, 2018 19:44
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

In reading this guide, I think I'll make a follow-up PR to do some general editing around the file as well.

- [Dashboard](#dashboard-schematic)
- [Navigation](#navigation-schematic)
- [Table](#table-schematic)
- [Tree](#tree-schematic)
Copy link
Member

Choose a reason for hiding this comment

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

Could you change this to a table where each row has the name and a brief description?

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated. I tried my best explaining the schematics in a short but accurate way. Also, yeah I agree that it would make sense to improve that whole document at some point.

@jelbourn jelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed action: merge The PR is ready for merge by the caretaker labels Oct 10, 2018
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn added the action: merge The PR is ready for merge by the caretaker label Oct 10, 2018
@andrewseguin andrewseguin merged commit c14c34e into angular:master Oct 10, 2018
roboshoes pushed a commit to roboshoes/material2 that referenced this pull request Oct 23, 2018
…13520)

* docs: show all available schematics in the schematics guide

* Updates the outdated `schematics` guide to mention the new available component schematics.

* Address feedback
@angular-automatic-lock-bot
Copy link

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.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement docs This issue is related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants