Let's jump right in.
PROBLEM
Customer needs a price range search feature so that one could look between $5 and $10
I see that in the SearchCatalog, price ranges are added to the SearchWithProvider call like so,
As an array of strings. So I figured I would add an option that built a string for them, and pushed it to the PriceRanges array. ["$15000 - $19999"], for example.
It did not work as expected. My first step to debug was to look at what the network call itself returned,
Interestingly I see that it returns an array of five PriceRanges. This meant that this is data coming somewhere from our API, so I went searching in CEF. I figured I would search for ' "$1000 ' to see what file this was being handled in
I'm led to the "ProductSearchModule.cs" file in the searching provider. Sure enough, there is a hard-coded set of five --
For this specific problem, it seems our implementation of ElasticSearch isn't able to handle a dynamic price range -- but we still want to offer an easier way to change these five ranges rather than directly editing code.
SOLUTION
We'll wrap all of these in appSettings. To create a new appSetting, go to CEF\00.Clarity.Ecommerce.RegistryLoader\JSConfigs\CEFConfig.Properties.cs
From there, you should be able to copy-paste most of what you'll need to do,
Note the default values correspond to the hard-coded strings in our previous files.
Next, we'll replace those hard-coded strings with our appSettings
NEXT!
Great! You fixed this bug on your local. Someone else is going to have to do that when this comes up... or you could create a pull request into the latest core branch of CEF.
To do this, have a CEF local set up and built on the 'develop' branch, identify the files you edited and make the same edits.
Be sure your codebase is fully up to date,
Switch to a new branch
Make your edits,
Be sure it builds
Now pull request
First, push your code,
Be sure to follow the naming convention. [PROJECT] Merge <branch> into <branch>
Also, be sure to link a work item.
You have added to CEF. The machine spirits are pleased.