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

Conversation

devversion
Copy link
Member

@devversion devversion commented Jul 23, 2016

  • Currently the md-require-match attribute checks whether an item is selected or not. The validity shouldn't be set to false, if the searchText is empty.

Closes #9072.

Breaking Change: The autocomplete validator md-require-match no longer matches if the search text is empty

@devversion devversion added the needs: review This PR is waiting on review from the team label Jul 23, 2016
@gpopovic
Copy link

gpopovic commented Aug 5, 2016

Any ETA when this gets accepted?

@devversion
Copy link
Member Author

Not sure @gpopovic - It's a pretty simple update.

@EladBezalel
Copy link
Member

LGTM 👍

…xt is provided

* Currently the `md-require-match` attribute checks whether an item is selected or not. The validity shouldn't be set to false, if the searchText is empty.

Closes angular#9072.

Breaking Change: The autocomplete validator `md-require-match` no longer matches if the search text is empty
@devversion devversion force-pushed the update/autocomplete-require-match-empty branch from 783bce8 to 702a55b Compare September 2, 2016 21:23
@topherfangio
Copy link
Contributor

LGTM 👍

@topherfangio
Copy link
Contributor

@ThomasBurleson Ready for your review.

@ThomasBurleson ThomasBurleson added needs: presubmit and removed needs: review This PR is waiting on review from the team labels Sep 3, 2016
@ThomasBurleson ThomasBurleson added this to the 1.1.2 milestone Sep 3, 2016
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels Sep 8, 2016
@kara kara merged commit 399016d into angular:master Sep 8, 2016
@devversion devversion deleted the update/autocomplete-require-match-empty branch September 8, 2016 00:13
Frank3K pushed a commit to Frank3K/material that referenced this pull request Sep 11, 2016
…xt is provided (angular#9119)

* Currently the `md-require-match` attribute checks whether an item is selected or not. The validity shouldn't be set to false, if the searchText is empty.

Closes angular#9072.

Breaking Change: The autocomplete validator `md-require-match` no longer matches if the search text is empty
@ekarudianto
Copy link

Please review and release and update soon :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants