Skip to content

Conversation

TedSander
Copy link
Contributor

Add a pending state to AbstractControl and a function to set that state on
themselves and their parents. This will be used for both individual async
validators and when the imperitive mode is used. Design Doc

Add a pending state to AbstractControl and a function to set that state on
themselves and their parents. This will be used for both individual async
validators and when the imperitive mode is used. [Design Doc](https://docs.google.com/document/d/1EnJ3-_iFpVKFz1ifN1LkXSGQ7h3A72OQGry2g8eo7IA/edit?pli=1#heading=h.j53rt81eegm4)
@TedSander
Copy link
Contributor Author

@mhevery @vsavkin This changes the public API to add a pending state to AbstractControls as Victor and I talked about. Please take a detailed look and ensure that the API changes I have are acceptable. Feedback appreciated.

@TedSander TedSander added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 17, 2015
@TedSander
Copy link
Contributor Author

@vsavkin Any comment on this PR?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this the same as this._parent.markAsPending() ?

@vsavkin
Copy link
Contributor

vsavkin commented Oct 26, 2015

Merged c9fba3f

@vsavkin vsavkin closed this Oct 26, 2015
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants