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

fix(typeahead): Add formcontrol binding. Fixes #92 (first problem) #396

Closed
wants to merge 1 commit into from
Closed

fix(typeahead): Add formcontrol binding. Fixes #92 (first problem) #396

wants to merge 1 commit into from

Conversation

slintes
Copy link
Contributor

@slintes slintes commented Apr 11, 2016

all credits go to nrehm: #92 (comment)

@fdu-axiometrics
Copy link

can we merge this to solve the ngControl issue?

@Jeetah
Copy link

Jeetah commented May 18, 2016

This bug is really annoying since it prevents using ngFormControl with TypeAhead completely. Could someone please merge it?

@ghost
Copy link

ghost commented May 19, 2016

Im facing the same bug here and there is a solution written, merge please?

@valorkin
Copy link
Member

no worries it will be done

@Dinistro
Copy link
Contributor

@slintes There are issues in this fix. Yes, it makes using of ngControl possible, but it crashes if you try to use a typeahead without ngControl.

Unhandled Promise rejection: Template parse errors:
Cannot instantiate cyclic dependency! NgModel

@marcin992
Copy link

+1

@valorkin
Copy link
Member

valorkin commented Jun 6, 2016

Seems I found a convenient way how to support both of NgModel and NgControl
finishing with docs and datepicker popup and will get here

@Dinistro
Copy link
Contributor

Dinistro commented Jul 6, 2016

New forms and new typeahead implementation.

@Dinistro Dinistro closed this Jul 6, 2016
@slintes slintes deleted the typeahead_fix_no-provider branch August 29, 2016 19:48
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.

6 participants