Skip to content

docs(router): Fix typo in https://angular.io/guide/router#activated-route#38034

Closed
rnlee0054 wants to merge 1 commit into
angular:masterfrom
rnlee0054:patch-1
Closed

docs(router): Fix typo in https://angular.io/guide/router#activated-route#38034
rnlee0054 wants to merge 1 commit into
angular:masterfrom
rnlee0054:patch-1

Conversation

@rnlee0054

Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

In angular.io, it linked to the wrong part of the page. https://angular.io/guide/router#activated-route

Issue Number: N/A

What is the new behavior?

It should now link to the right header.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@pullapprove pullapprove Bot requested a review from atscott July 13, 2020 22:46
@atscott atscott added comp: docs target: patch This PR is targeted for the next patch release labels Jul 13, 2020
@ngbot ngbot Bot modified the milestone: needsTriage Jul 13, 2020
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Jul 13, 2020
@atscott

atscott commented Jul 13, 2020

Copy link
Copy Markdown
Contributor

Hi @rnlee0054, thanks for the fix. Could you amend the commit message header to be "docs(router)..." to conform to our header requirements (and get the lint check to pass)?

@rnlee0054 rnlee0054 changed the title docs: Fix typo in https://angular.io/guide/router#activated-route docs(router): Fix typo in https://angular.io/guide/router#activated-route Jul 14, 2020
@rnlee0054

Copy link
Copy Markdown
Contributor Author

Done. thanks !

@atscott atscott closed this in 4ec3026 Jul 14, 2020
atscott pushed a commit that referenced this pull request Jul 14, 2020
@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 Aug 14, 2020
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants