Skip to content

Added value field #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added value field #42

wants to merge 1 commit into from

Conversation

shotap
Copy link

@shotap shotap commented Jun 30, 2015

now ac-options supports value fields like ac-options="l.name as l.id in loadData()"

now ac-options supports value fields like `ac-options="l.name as l.id in loadData()"`
@shotap
Copy link
Author

shotap commented Jul 28, 2015

??!?

@john-oc
Copy link
Owner

john-oc commented Aug 25, 2015

@shotap - apologies for the delay on this one. The day-job has been a bit hectic, but try to will test and merge this soon.

@john-oc
Copy link
Owner

john-oc commented Aug 25, 2015

Having a look at this now, but just wondering what these lines are for?

                $scope.initialSelection = angular.copy($scope.model);

                if (!isNaN($scope.initialSelection))
                    $scope.initialSelection = $scope.initialSelection * 1;

John

@shotap
Copy link
Author

shotap commented Aug 26, 2015

It because

'1' !=== 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants