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

Added a link for further info about Angular's DI on Tutorial, Step 7 - Routing & Multiple Views #7007

Closed
wants to merge 1 commit into from

Conversation

thicolares
Copy link
Contributor

Request Type: docs

How to reproduce: Visite the step 7 of the tutorial:
http://code.angularjs.org/1.3.0-beta.4/docs/tutorial/step_07

After reading "A Note About DI, Injector and Providers", DI on Angular was so clear to me.

Component(s):

Impact:

Complexity:

This issue is related to:

Detailed Description:

This article is fantastic and really helped on understanding how DI works on Angular. It may be useful to other beginners -- because, at first glance, this topic (DI on Angular) ended a little bit hazy for me.

Other Comments:

This article is fantastic and really helped on understanding how DI works on Angular. It may be useful to other beginners.
@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 (#7007)

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!

@thicolares thicolares changed the title Added a link for further info about Angular's DI Added a link for further info about Angular's DI on Tutorial, Step 7 - Routing & Multiple Views Apr 5, 2014
@@ -98,6 +98,10 @@ of configuring the injector. As opposed to AMD or require.js modules, Angular mo
solve the problem of script load ordering or lazy script fetching. These goals are totally independent and
both module systems can live side by side and fulfil their goals.

For deepen your understanding of DI on Angular, see
[how Angular's dependency injection system works](git://github.com/angular/angular.js.wiki.git),
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you've added the wrong URL here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, gosh. I lost my first pull request. It was the second. I'll fix it. The correct link was:
https://github.com/angular/angular.js/wiki/Understanding-Dependency-Injection

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry about the mess. I've pulled again: #7010 (instead of correct last commit)

@thicolares thicolares closed this Apr 5, 2014
@thicolares thicolares deleted the patch-3 branch April 5, 2014 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants