feat(router): support deep-linking to anywhere in the app - #2772

Still working on this, but early feedback welcome. Hoping to finish by EOD.

Review status: 0 of 10 files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

^{modules/angular2/src/router/path_recognizer.ts, line 159 [r1] (raw file):}
can you type the handler symbol and then use handler.component and handler.type instead of reflective calls? or is there some other reason (other than lack of type info) for doing it this way?

^{modules/angular2/src/router/path_recognizer.ts, line 187 [r1] (raw file):}
trailing slash? I though we said that we wouldn't do trailing slashes

^{modules/angular2/src/router/route_registry.ts, line 153 [r1] (raw file):}
linkDsl => linkParams?

^{modules/angular2/src/router/route_registry.ts, line 172 [r1] (raw file):}
what do you need to do with params here? I thought that it was the componentRecognizer.generate that took care of params

^{modules/angular2/src/router/router_link.ts, line 41 [r1] (raw file):}
isn't this linkParams rather than route?

^{modules/angular2/test/router/outlet_spec.ts, line 197 [r1] (raw file):}
:-)

Comments from the review on Reviewable.io

Review status: 0 of 15 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

^{modules/angular2/src/router/path_recognizer.ts, line 187 [r1] (raw file):}
This is joining on /, not joining with / at the end.

^{modules/angular2/src/router/route_registry.ts, line 172 [r1] (raw file):}
This TODO is totally out of date.

Comments from the review on Reviewable.io

^{modules/angular2/test/router/outlet_spec.ts, line 197 [r1] (raw file):}
?

Comments from the review on Reviewable.io

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.}