X Tutup
Skip to content

feat(router): support links with just auxiliary routes#5930

Closed
btford wants to merge 1 commit intoangular:masterfrom
btford:fix-router-aux-route-things
Closed

feat(router): support links with just auxiliary routes#5930
btford wants to merge 1 commit intoangular:masterfrom
btford:fix-router-aux-route-things

Conversation

@btford
Copy link
Copy Markdown
Contributor

@btford btford commented Dec 15, 2015

Support links like: <a [routerLink]="['/', ['Modal']]">open modal</a> With no primary route part.

@btford btford added feature Label used to distinguish feature request from other issues comp: router action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 15, 2015
@btford btford force-pushed the fix-router-aux-route-things branch from 9aaed6c to 1db585b Compare December 15, 2015 23:41
@btford btford force-pushed the fix-router-aux-route-things branch from 1db585b to 9fd6745 Compare December 15, 2015 23:49
@btford
Copy link
Copy Markdown
Contributor Author

btford commented Dec 16, 2015

I followed @matsko home to get him to LGTM it in person and he did. 👹

@btford btford added pr_state: LGTM 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 Dec 16, 2015
@mary-poppins
Copy link
Copy Markdown

User @mprobst does not have PR merging privileges.

@mprobst
Copy link
Copy Markdown
Contributor

mprobst commented Dec 16, 2015

@alexeagle I need that privilege, apparently.

@mary-poppins
Copy link
Copy Markdown

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

@btford
Copy link
Copy Markdown
Contributor Author

btford commented Dec 16, 2015

👏 thanks @mary-poppins and @mprobst

@btford btford closed this in 2a2f9a9 Dec 16, 2015
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 7, 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 feature Label used to distinguish feature request from other issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup