Skip to content
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

rating component does not support reactive forms (requires ngModel) #959

Closed
born2net opened this issue Sep 12, 2016 · 6 comments
Closed

Comments

@born2net
Copy link

born2net commented Sep 12, 2016

rating component does not support reactive forms (requires ngModel)
please change selector to <rating instead of [ngModel]

It implements ControlValueAccessor, but never provides NG_VALUE_ACCESSOR so that implementation is useless.

tx

Sean

Angular 2 Kitchen sink: http://ng2.javascriptninja.io
and source@ https://github.com/born2net/ng2Boilerplate

@born2net
Copy link
Author

@Martin-Luft
Copy link
Contributor

The following ng2-bootstrap components use the ngModel selector binding:

pagination.component
pager.component
datepicker.component
datepicker-popup.component
button-checkbox.directive
button-radio.directive
timepicker.component
rating.component

@Martin-Luft
Copy link
Contributor

What do you mean with

selector to <rating

Is the solution to use ngControl binding instead of ngModel binding?

@born2net
Copy link
Author

if you look at the component, it uses ngModel, but reactive forms do not use ngModel, thus the directive does not bind to the component element.

@Martin-Luft
Copy link
Contributor

#1023

@valorkin
Copy link
Member

valorkin commented Dec 7, 2016

fixed with #298

@valorkin valorkin closed this as completed Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants