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

docs(tutorial): Update step_12.ngdoc #5156

Closed
wants to merge 1 commit into from

Conversation

elwinarens
Copy link
Contributor

  1. The example includes the wrong version of jQuery, if the version notification is still correct and up-to-date.
  2. If jQuery 1.10.x must be used, then i think it's important to move the version alert up to below the example of the app/index.html, instead of way down below the fold.

@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

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!

@elwinarens
Copy link
Contributor Author

Signed CLA as Elwin Arens

@elwinarens elwinarens closed this Nov 27, 2013
@elwinarens elwinarens deleted the patch-1 branch November 27, 2013 07:47
@elwinarens elwinarens restored the patch-1 branch November 27, 2013 07:49
@elwinarens elwinarens reopened this Nov 27, 2013
@elwinarens
Copy link
Contributor Author

Not sure why the build is failing.

@elwinarens elwinarens deleted the patch-1 branch December 4, 2013 22:55
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
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.

2 participants