homeflowjs 0.10.10 → 0.10.11

package/actions/branches.actions.js

@@ -1,5 +1,10 @@
 import BranchesActionTypes from './branches.types';
 
+export const setBranches = (payload) => ({
+  type: BranchesActionTypes.SET_BRANCHES,
+  payload,
+});
+
 export const setBranchesSearch = (payload) => ({
   type: BranchesActionTypes.SET_BRANCHES_SEARCH,
   payload,

package/actions/branches.types.js

@@ -1,4 +1,5 @@
 const BranchesActionTypes = {
+  SET_BRANCHES: 'SET_BRANCHES',
   SET_BRANCHES_SEARCH: 'SET_BRANCHES_SEARCH',
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homeflowjs",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "description": "JavaScript toolkit for Homeflow themes",
   "main": "index.js",
   "scripts": {

package/properties/properties-display/properties-display.component.jsx

@@ -58,10 +58,10 @@ const PropertiesDisplay = ({
          */
         if (inserts && (index % inserts[0].frequency) === 0 && index !== 0) {
           return (
-            <>
+            <React.Fragment key={property.property_id}>
               {inserts[0].component}
-              <Item key={property.property_id} property={property} {...other} />
-            </>
+              <Item property={property} {...other} />
+            </React.Fragment>
           );
         }
         return (

package/properties/stamp-duty-calculator/default-calculator.component.jsx

@@ -31,7 +31,7 @@ const DefaultCalculator = ({ purchasePrice, setPurchasePrice, firstTimeBuyer, ad
     }
 
     const {
-      rate1, 
+      rate1,
       rate2,
       rate3,
       rate4,
@@ -56,7 +56,7 @@ const DefaultCalculator = ({ purchasePrice, setPurchasePrice, firstTimeBuyer, ad
       setPurchasePrice(property.price_value);
       calculateStampDuty(property.price_value);
     }
-  }, []);
+  }, [firstTimeBuyer, additionalProperty]);
 
   const handlePriceInputChange = ({ target: { value } }) => {
     const numberString = value.substring(1);
@@ -102,7 +102,7 @@ const DefaultCalculator = ({ purchasePrice, setPurchasePrice, firstTimeBuyer, ad
         </p>
 
         {(firstTimeBuyer && property.price_value >= 625000) && (
-          <p id='first_time_buyer_info'>First time buyers purchasing property for more than £625,000 will not be entitled to any relief and will pay SDLT at the standard rates.</p>
+          <p id="first_time_buyer_info">First time buyers purchasing property for more than £625,000 will not be entitled to any relief and will pay SDLT at the standard rates.</p>
         )}
 
         <div className="results-table-wrap">

package/reducers/branches.reducer.js

@@ -1,11 +1,17 @@
 import BranchesActionTypes from '../actions/branches.types';
 
 const INITIAL_STATE = {
+  branches: [],
   branchesSearch: '',
 };
 
 const branchesReducer = (state = INITIAL_STATE, action) => {
   switch (action.type) {
+    case BranchesActionTypes.SET_BRANCHES:
+      return {
+        ...state,
+        branches: action.payload,
+      };
     case BranchesActionTypes.SET_BRANCHES_SEARCH:
       return {
         ...state,

package/reducers/branches.reducer.test.js

@@ -0,0 +1,10 @@
+import branchesReducer from './branches.reducer';
+
+describe('branchesReducer', () => {
+  it('Sets the branches when it receives the SET_BRANCHES action', () => {
+    const state = { branches: [] };
+    const reducedState = branchesReducer(state, { type: 'SET_BRANCHES', payload: [1, 2, 3] });
+
+    expect(reducedState.branches).toEqual([1, 2, 3]);
+  });
+});