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

Improved advanced form documentation example #4320

Closed
wants to merge 1 commit into from
Closed

Improved advanced form documentation example #4320

wants to merge 1 commit into from

Conversation

jbruni
Copy link
Contributor

@jbruni jbruni commented Oct 8, 2013

Pass the $index directly to the removeContact function, instead of searching for it in an expensive forEach loop.

@jbruni
Improved advanced form documentation example
a777563 
Pass the `$index` directly to the `removeContact` function, instead of searching for it in an expensive forEach loop.
@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!

jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
@jbruni @jamesdaily
docs(cookbook/adv_forms): simplify example
e9d8f62 
Pass the `$index` directly to the `removeContact` function, instead of
searching for it in an expensive forEach loop.

Closes angular#4320
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
@jbruni @jamesdaily
docs(cookbook/adv_forms): simplify example
f3b7426 
Pass the `$index` directly to the `removeContact` function, instead of
searching for it in an expensive forEach loop.

Closes angular#4320
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Post-1.2
Development

Successfully merging this pull request may close these issues.
2 participants
@jbruni @mary-poppins