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

docs(select): Updated ngOptions track by examples #10224

Closed
wants to merge 2 commits into from

Conversation

SteveShaffer
Copy link
Contributor

Made the example shown consistent with the advice above it regarding not using select as and track by in the same comprehension expression. Also changed references to trackexpr to track by since trackexpr is not defined except in the examples.

Made the example shown consistent with the advice above it regarding not using `select as` and `track by` in the same comprehension expression.  Also changed references to `trackexpr` to `track by` since `trackexpr` is not defined except in the examples.
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@SteveShaffer
Copy link
Contributor Author

Signed the CLA

@googlebot
Copy link

CLAs look good, thanks!

The documentation for ngRepeat includes such an example specifying the proper order for filters and and "track by" clauses in the comprehension expression, but these docs for ngOptions do not.
@SteveShaffer
Copy link
Contributor Author

Addresses confusion in #10225

@lgalfaso
Copy link
Contributor

lgalfaso commented Dec 1, 2014

Hi @WatchSteveDrum, thanks for the PR. Can you please squash the two commits into one?

@lgalfaso lgalfaso added this to the 1.3.6 milestone Dec 1, 2014
@lgalfaso
Copy link
Contributor

lgalfaso commented Dec 3, 2014

landed as b9e5eaf

@lgalfaso lgalfaso closed this Dec 3, 2014
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