Conversation
ekraffmiller
left a comment
There was a problem hiding this comment.
Looks good. Do we need a validation test in the getCollectionItems use case that checks if searchText is defined, when order is set to SCORE?
Not sure I would do that, the API is not complaining about that, if you send |
|
@ekraffmiller this is the SPA side PR using this PR version in case you want to review it also. |
|
ekraffmiller
left a comment
There was a problem hiding this comment.
looks good, approved
|
Tests are passing - merging PR |
What this PR does / why we need it:
This just adds a new
SCOREoption to theSortTypeenumWhich issue(s) this PR closes:
Suggestions on how to test this:
Code review and tests passing.