Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Update step_07.ngdoc to fix typo in route pattern #7313

Closed
wants to merge 1 commit into from
Closed

Update step_07.ngdoc to fix typo in route pattern #7313

wants to merge 1 commit into from

Conversation

jamiekrug
Copy link
Contributor

One instance of /phones/:phoneId erroneously had a singular version, /phone/:phoneId, which does not match what was actually used in the code.

One instance of `/phones/:phoneId` erroneously had a singular version, `/phone/:phoneId`, which does not match what was actually used in the code.
@mary-poppins
Copy link

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

  • Uses the issue template (#7313)

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@jamiekrug
Copy link
Contributor Author

I started to fill out the issue template for this, but the "Login to Update Issue" authorization requests read and write access to all public and private repositories, which makes me very uneasy. This is a single-letter typo fix to documentation. I'd like to help out and follow helpful work flows, but that issue template authorization seems way overreaching, no? Thanks!

@jamiekrug jamiekrug added cla: yes and removed cla: no labels May 1, 2014
@petebacondarwin
Copy link
Contributor

No problem! Thanks for the fix. We don't need all this for a simple docs fix.

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.

3 participants