Skip to content

Conversation

@vsavkin
Copy link
Contributor

@vsavkin vsavkin commented Apr 25, 2016

No description provided.

@vsavkin vsavkin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 25, 2016
@vsavkin vsavkin force-pushed the support_aux_routes branch from 80b6a71 to bc030f6 Compare April 25, 2016 23:57
Copy link
Contributor

Choose a reason for hiding this comment

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

Make "aux" an overridable default?

@vsavkin vsavkin force-pushed the support_aux_routes branch from bc030f6 to 4affc9a Compare April 27, 2016 22:37
@HostBinding() private href: string;

constructor(private _router: Router, private _segment: RouteSegment) {
this._subscription = ObservableWrapper.subscribe(_router.changes, (_) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove the wrapper

@vsavkin vsavkin force-pushed the support_aux_routes branch from 9e01219 to 17d1e1b Compare April 29, 2016 00:46
@vsavkin vsavkin changed the title Alt Router: Support aux routes Alt Router: Support aux routes/RouterLink/Location Apr 29, 2016
@vsavkin vsavkin force-pushed the support_aux_routes branch from 17d1e1b to 143fd1e Compare April 29, 2016 00:47
@vsavkin
Copy link
Contributor Author

vsavkin commented Apr 29, 2016

Did a walkthrough with Misko

@vsavkin vsavkin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 29, 2016
@mary-poppins
Copy link

Merging PR #8220 on behalf of @mprobst to branch presubmit-mprobst-pr-8220.

@mprobst
Copy link
Contributor

mprobst commented Apr 29, 2016

Woohoo @vsavkin !

@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 8, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants