Skip to content

Conversation

aChanEP
Copy link
Contributor

@aChanEP aChanEP commented Sep 30, 2019

Description:

Pull request integrates the more like this endpoint from bloomreach: https://help.bloomreach.com/display/BRINT/More+like+this+%28MLT%29+API

If the bloomreach settings are turned on the recommended items are going to be powered by bloomreach.

Linting:

  • No linting errors

Component Updates:

This will be part of PR #428 so can update the component number there.

Tests:

  • E2E tests (npm test run with e2e)
  • Manual tests

1.) To test turn on the bloomreach settings

2.) Fill in the new recommended items: mlt config.

"productRecommendationConfig": {
        "requestType": "request_type=mlt",
        "rows": "rows=10",
        "start": "start=0",
        "facetLimit": "facet.limit=20",
        "fl": "fl=pid%2Ctitle%2Cbrand%2Cprice%2Csale_price%2Cpromotions%2Cthumb_image%2Csku_thumb_images%2Csku_swatch_images%2Csku_color_group%2Curl%2Cprice_range%2Csale_price_range%2Cdescription"
      }

3.) notice that the component has been swapped and the products that are being recommended are coming from bloomreach.

4.) turn on storybook and notice that the recommended bloomreach product is also in storybook.

NOTE: Merge this PR before #428 as it is branched off of #428.

Documentation:

  • Requires documentation updates

Able to get mlt products showing up in products display page
use async await pattern
Adjust the name of mlt function
get rid of prop types
fix typescript errors
Move state and props into new file
Able to place bloomreach product recommendations into storybook
Add product recommendation configs
add a new line
adjust files to use new configurations properly
Change the naming of the component
change the padding of the title
Fix the linking to product recommendations
revert settings to false
@aChanEP aChanEP merged commit 808afe2 into feature/bloomreach-search-results-page Oct 4, 2019
@aChanEP aChanEP deleted the feature/bloomreach-integration-mlt branch October 4, 2019 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants