Feature/warning before delete (#177)

* added before delete notification * typo
digirati-co-uk · Oct 23, 2018 · b071b19 · b071b19
1 parent 5b49888
commit b071b19
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/src/reducers/range.js b/src/reducers/range.js
@@ -260,7 +260,6 @@ const range = (state = DEFAULT_RANGES_STATE, action) => {
       const removals = {
         $unset: [],
       };
-      const deleted = [];
       Object.values(state)
         .sort(
           (a, b) =>

diff --git a/src/sagas/index.js b/src/sagas/index.js
@@ -174,8 +174,17 @@ function* afterDelete() {
 }
 
 function* multiDelete({ selecteds }) {
-  for (var i = 0; i < selecteds.length; i++) {
-    yield put(deleteRange(selecteds[i]));
+  let confirmed = true;
+  if (selecteds.length > 1) {
+    confirmed = yield call(
+      showConfirmation,
+      'Multiple ranges will be deleted. Redundant length groups will be removed. Do you wish to continue?'
+    );
+  }
+  if (confirmed) {
+    for (var i = 0; i < selecteds.length; i++) {
+      yield put(deleteRange(selecteds[i]));
+    }
   }
 }