-
Notifications
You must be signed in to change notification settings - Fork 15
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(app & service subscription): fix search text clearing #371
fix(app & service subscription): fix search text clearing #371
Conversation
Hi @MaximilianHauer, Can you please review this PR? |
@ss-nikunj kindly add changelog entry and fix dependency issue |
Hi @manojava-gk, I have added a changelog entry to the description ( as per @evegufy mentioned in one of my PR ). I will resolve the dependency issue |
…ops and pass to toolbar
45a3652
to
fafe401
Compare
Quality Gate passedIssues Measures |
lgtm but to make it perfect ensure that the changelog entry is embedded this makes it easy to copy paste it. |
Hi @MaximilianHauer, As you mentioned I have modified the changelog entry. |
Description
onClearSearch
method is implemented. However, the method was not being correctly passed down from the parent component, preventing it from functioning as expected.Fix
Issue
Checklist
Changelog Entry