-
Notifications
You must be signed in to change notification settings - Fork 14
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
Improved catalog filtering #2035
Conversation
c79b24a
to
1610399
Compare
1610399
to
4560f92
Compare
089dd43
to
b661f64
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!
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.
Some tiny stuff then we can merge 👍
b661f64
to
7def41a
Compare
7def41a
to
1a42780
Compare
Thanks @dottorblaster. Addressed the feedbacks! |
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
Description
This PR introduces the possibility to feed the
Select
component with a list of objects as options, where thevalue
anddisabled
properties can be defined.As a result disabled options can also be handled.
Currently unsupported filters have been disabled (target type host and cluster type hana scale out and ascs/ers)
Enabling disabled options needs a pr to do so.
Further automatism to rely on the catalog to determine enabled/disabled options needs further refactoring on the catalog state. Tracking an item in the backlog
How was this tested?
Automated tests/storybook.
https://2035.prenv.trento.suse.com/catalog
Catalog Story
Select Story