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
OCPBUGS-13359: Fix crash when filtering the quick start catalog #13126
OCPBUGS-13359: Fix crash when filtering the quick start catalog #13126
Conversation
@jerolimov: This pull request references Jira Issue OCPBUGS-13359, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@jerolimov: This pull request references Jira Issue OCPBUGS-13359, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/kind bug |
/label acknowledge-critical-fixes-only |
34a400a
to
2fe8f06
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.
Verified it and it works as expected.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jerolimov, vikram-raj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@jerolimov: This pull request references Jira Issue OCPBUGS-13359, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@jerolimov: This pull request references Jira Issue OCPBUGS-13359, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@jerolimov: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@jerolimov: Jira Issue OCPBUGS-13359: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-13359 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Fixes:
https://issues.redhat.com/browse/OCPBUGS-13359
Analysis / Root cause:
When the user enters a filter into the quick start catalog the catalog page crashes. This happened because the PatternFly Quick Start library uses a PF Search field, which changed the props 'recently': patternfly/patternfly-react#8516
So the Quick Start library and the used Search component aren't compatible, and the callback handler expects a string value but receives an event. But the same function is called with a string value when the user clicks the clear button.
A fix for the PatternFly Quick Start library wasn't merged yet: patternfly/patternfly-quickstarts#237
Solution Description:
Since the PF fix wasn't merged yet and this would require a PF Quick Start update, I decided to clone the
QuickStartCatalogPage
component from@patternfly/quickstarts
and add it to our code base. And fix the issue in the copied version.The difference to the origin version is mostly that the
onSearchInputChange
callback handles now strings and events:I opened a tech debt issue (ODC-7381) to update the Quick Start dependency in 4.15 from our current version 2.4.0 to the latest version 5.0.0. (We should do this as part of the PF 5 update.)
Screen shots / Gifs for design review:
Crash without this PR:
4-14-broken.mp4
No crash with this PR:
4-14-fixed.mp4
Unit test coverage report:
Untouched
Test setup:
Browser conformance: