feat: product category search #25
Conversation
There was a problem hiding this comment.
Skide godt @RAORelewise
Jeg er stødt på lidt små ting.
UPDATE: Jeg har kørt CI'en på din branch igen og der er en fejl :(
| test('ProductCategorySearch: Relevance modifier without conditions', async() => { | ||
|
|
||
| const request: ProductCategorySearchRequest = baseProductCategoryBuilder() | ||
| .relevanceModifiers(b => b.addProductDataRelevanceModifier('NoveltyBoostModifier', conditions => conditions, ValueSelectorFactory.dataDoubleSelector('NoveltyBoostModifier'))) |
There was a problem hiding this comment.
Giver denne relevance modifier mening at teste her eller er det bare copy paste fra en anden test? :)
There was a problem hiding this comment.
Det var bare for at have den med tog ikke lige højde for indholdet :)
Co-authored-by: Martin Zanoni <zanoniii@gmail.com>
Co-authored-by: Martin Zanoni <zanoniii@gmail.com>
Co-authored-by: Martin Zanoni <zanoniii@gmail.com>
…ewise/relewise-sdk-javascript into feat-product-category-search
|
@mzanoni Ah, kan se det er pga. vi har fjernet de her DataInteger facets, da de slet ikke "virker" og har aldrig virket. Så har fjernet typerne i api'et. De er nu også fjernet her i js sdket. Og så brokkede den sig over at "trackingNumber" er væk fra Order, så har omdøbt den til orderNumber, dog uden at gøre det i metoden for ikke at breake noget bagud |
|
@RAORelewise Kristoffer har fikset tracking nummer, det er også meget breaking at rename den fordi alle som bruger det fra CDN ikke vil opdate at det ikke virker mere. Vi fikser det bare ved at merge hans PR ind i din. |
@mzanoni Vi kan selvfølgelig ikke merge den ind før vi har publiceret søgningen i api'et