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

docs(router): Updated routing examples to use routing module #2478

Merged
merged 1 commit into from
Oct 5, 2016

Conversation

brandonroberts
Copy link
Contributor

@brandonroberts brandonroberts commented Sep 26, 2016

All examples updating to use separate "routing modules"
Routing tutorial simplified and prose updated
Router developer guide examples and prose updated

@brandonroberts brandonroberts force-pushed the routing-modules branch 5 times, most recently from aa17db4 to 3f30c91 Compare September 28, 2016 02:10
@brandonroberts brandonroberts changed the title WIP: docs(router): Updated routing examples to use routing module docs(router): Updated routing examples to use routing module Sep 28, 2016
@brandonroberts
Copy link
Contributor Author

brandonroberts commented Sep 28, 2016

@wardbell @johnpapa this PR is ready for review. All the samples have been updated to use Routing Modules. I didn't change any of the prose in the NgModule chapter but I did update the routing examples.

@@ -2,7 +2,7 @@

block includes
include ../_util-fns
- var _appRoutingTsVsAppComp = 'app.routing.ts'
- var _appRoutingTsVsAppComp = 'app.module.ts'
Copy link
Contributor

Choose a reason for hiding this comment

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

what does this jade syntax do? If its code, we dont use underscore prefixes by convention. If its styling, ignore this :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Local Jade variable names used to render TS and Dart prose from the same source. Such variable names do not appear in the rendered HTML.

Copy link
Contributor

Choose a reason for hiding this comment

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

perfect. thanks

@@ -229,7 +229,7 @@ block get-heroes-details
## Update hero details

We can edit a hero's name already in the hero detail view. Go ahead and try
it. As we type, the hero name is updated in the view heading.
it. As we type, the hero name is updated in the view heading.
Copy link
Contributor

Choose a reason for hiding this comment

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

@wardbell mentioned to me a few days ago that we are switching from the "we" to the "you" in our prose. If so, this needs to switch. But check with Ward.

Copy link
Contributor

Choose a reason for hiding this comment

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

We are switching ot you. But this switch is just one part of a deliberate sweep of the docs leading to conformance with the Google docs standards. No need for Brandon to take that on for this particular revision.

After a doc has been converted, then our revisions must conform.

@brandonroberts
Copy link
Contributor Author

@wardbell @johnpapa I updated the ngmodule guide and ngmodule faq cookbook with sections for routing modules. Its review for complete review

@wardbell
Copy link
Contributor

wardbell commented Oct 3, 2016

This PR responds to the concern in issue #2527 by deleting the offending section

Simplified routing in tutorial example

Updated ngmodule guide and ngmodule faq with routing module prose
@wardbell wardbell merged commit ff11881 into angular:master Oct 5, 2016
@wardbell wardbell deleted the routing-modules branch October 5, 2016 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants