-
Notifications
You must be signed in to change notification settings - Fork 19
[OcSelect] Fix search for options provided as objects #1602
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
2afc98a
to
ff9dcbe
Compare
only add fuse keys property if label ist set
ff9dcbe
to
e30830f
Compare
@dschmidt can you please have a look if this works for you |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Yep, works for me. Feel free to merge if you're happy now (I just cannot approve my own PR) |
@@ -179,19 +179,19 @@ describe("OcTableFiles", () => { | |||
}) | |||
}) | |||
|
|||
describe("resource details", () => { | |||
test.skip("resource details", () => { |
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.
test.skip
? 🤔
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.
ah, because they were not enabled before
🎉 |
Description
I made
OcSelect.filter
make use of thelabel
prop as the original filter function of vue-select also does, i.e. this is a regression fix for a breaking change introduced in #1521Related Issue
Motivation and Context
This fixes broken search in ownBrander 2.0.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: