Replaced lodash.merge with merge-options (#312)

mapbox · Jan 16, 2024 · bd29cf7 · bd29cf7
1 parent 15d305d
commit bd29cf7
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 20 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -68,11 +68,10 @@
   },
   "dependencies": {
     "@mapbox/polyline": "^1.1.1",
-    "deep-assign": "^3.0.0",
     "lodash.debounce": "^4.0.6",
     "lodash.isequal": "^4.2.0",
-    "lodash.merge": "^4.6.2",
     "lodash.template": "^4.2.5",
+    "merge-options": "^3.0.4",
     "redux": "^4.2.0",
     "redux-thunk": "^2.4.2",
     "suggestions": "^1.7.1",

diff --git a/src/reducers/index.js b/src/reducers/index.js
@@ -1,7 +1,7 @@
 import * as types from '../constants/action_types.js';
-import merge from 'lodash.merge';
+import merge from 'merge-options';
 
-const initialState = {
+const getInitialState = () => ({
   // Options set on initialization
   api: 'https://api.mapbox.com/directions/v5/',
   profile: 'mapbox/driving-traffic',
@@ -49,9 +49,9 @@ const initialState = {
   fetchDirectionsRequest: null,
   routeIndex: 0, 
   routePadding: 80
-};
+});
 
-function data(state = initialState, action) {
+function data(state = getInitialState(), action) {
   switch (action.type) {
   case types.SET_OPTIONS: {
     return merge({}, state, action.options);