-
Notifications
You must be signed in to change notification settings - Fork 6.8k
fix(material/chips): ensure that edit input query is re-evaluated on time #27465
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
Conversation
6776a67
to
6df6e2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I added the dev-app-preview label so we can test this.
It's hard for me to verify this because I haven't been able to reproduce on any of our examples.
Deployed dev-app for d121ddc to: https://ng-dev-previews-comp--pr-angular-components-27465-dw5loip3.web.app Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt. |
I couldn't reproduce it against our dev app either. I was testing it in the internal demo. |
…time Fixes an issue that showed up in an internal app where the edit input query wasn't being evaluated on time which led to a runtime error. Fixes angular#27462.
6df6e2c
to
d121ddc
Compare
…time (angular#27465) Fixes an issue that showed up in an internal app where the edit input query wasn't being evaluated on time which led to a runtime error. Fixes angular#27462.
…time (angular#27465) Fixes an issue that showed up in an internal app where the edit input query wasn't being evaluated on time which led to a runtime error. Fixes angular#27462.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Fixes an issue that showed up in an internal app where the edit input query wasn't being evaluated on time which led to a runtime error.
Fixes #27462.