Merge pull request #154 from SailingSteve/steveStaffClientMar29-845am

Import DonorBox csv report

Author: DaleMcGrew

.eslintrc

@@ -3,7 +3,6 @@
   "parserOptions": {
     "sourceType": "module"
   },
-
   "env": {
     "browser": true,
     "node": true,
@@ -50,7 +49,6 @@
     "quotes": [1, "single", "avoid-escape"],
     "radix": 0,                                   // Dec 2018: We always use base 10, so specifying it each time seems excessive
     "react-hooks/exhaustive-deps": "warn",         // Checks effect dependencies
-    "react-hooks/rules-of-hooks": "error",         // Checks rules of Hooks
     "react/button-has-type": 1,
     "react/jsx-one-expression-per-line": 0,
     "react/destructuring-assignment": 0,          // Dec 2018: We should do this!  But right now we have 3990 warnings/errors if enabled.

package-lock.json

@@ -58,10 +58,10 @@
     "babel-plugin-styled-components": "^2.1.4",
     "chromatic": "^10.1.0",
     "clean-webpack-plugin": "^4.0.0",
-    "copy-webpack-plugin": "^12.0.2",
+    "copy-webpack-plugin": "^14.0.0",
     "cross-env": "^10.1.0",
-    "css-minimizer-webpack-plugin": "^6.0.0",
-    "eslint": "^7.24.0",
+    "css-minimizer-webpack-plugin": "^8.0.0",
+    "eslint": "^7.32.0",
     "eslint-config-airbnb": "^18.2.1",
     "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-jsx-a11y": "^6.4.1",
@@ -73,7 +73,7 @@
     "file-saver": "^2.0.5",
     "html-webpack-plugin": "^5.6.0",
     "jsdom": "^23.0.1",
-    "mocha": "^10.2.0",
+    "mocha": "^10.8.2",
     "pre-commit": "^1.0.10",
     "react-papaparse": "^4.4.0",
     "terser-webpack-plugin": "^5.1.2",

package.json

@@ -5,7 +5,6 @@ import { blurTextFieldAndroid, focusTextFieldAndroid } from '../../utils/cordova
 import { renderLog } from '../../utils/logging';
 import SearchBase from './SearchBase';
 
-/* eslint-disable jsx-a11y/control-has-associated-label  */
 class SearchBar2024 extends Component {
   constructor (props) {
     super(props);

src/js/common/components/Search/SearchBar2024.jsx

@@ -32,15 +32,15 @@ class SearchBase extends React.Component {
         this.props.onFocus(event);
       }
     });
-  }
+  };
 
   handleClear = () => {
     this.setState({ searchText: '' }, () => {
       if (this.props.onClear) {
         this.props.onClear();
       }
     });
-  }
+  };
 
   render () {
     return (

src/js/common/components/Search/SearchBase.jsx

@@ -20,7 +20,6 @@ glob('./build/main.*.js', (err, files) => {
     // fs.renameSync(file0, destName);
     // console.log(`WebPackPostBuild:  ${file0} has been renamed to ${destName}`);
     // next rename the <script defer="defer" src="/main.9871eda3cd9fbb2d224e.js"> in the generated index.html file
-    // eslint-disable-next-line consistent-return
     //     const indexFile = './build/index.html';
     //     fs.readFile(indexFile, 'utf8', (err2, data) => {
     //       if (err2) return console.log('read error: ', err);

src/js/common/node/webPackPostBuild.js

@@ -11,7 +11,6 @@ let polyfillsLoaded;
 
 // Copy of this function also in ./appleSiliconUtils, but moved here to fix dependency cycle problem
 function dumpObjProps (name, obj) {
-  // eslint-disable-next-line guard-for-in
   Object.keys(obj).forEach((key) => console.log(`Dump Object ${name} ${key}: ${obj[key]}`));
 }
 
@@ -337,7 +336,6 @@ export function isIPad () {
         width: window.screen.width * ratio,
         height: window.screen.height * ratio,
       };
-      /* eslint-disable no-extra-parens */
       if ((screen.width === 768 && screen.height === 1024) ||  // iPad, 9.7" 2010 and Gen 2, 2011 and Mini 2012
           (screen.width === 1536 && screen.height === 2048) || // iPad, 9.7" Gen 3 2012, Gen 4 2013, 2018 iPad, iPad Pro 2016, iPad Air 2013, and Mini Retina 2013
           (screen.width === 1668 && screen.height === 2224) || // iPad Pro 10.5" Gen 2  2017
@@ -675,7 +673,6 @@ export function restoreStylesAfterCordovaKeyboard (callerString) {
       if (isIOS()) {
         window.Keyboard.disableScroll(false);
       }
-      // eslint-disable-next-line no-empty
     } catch (e) {
       console.log('error in restoreStylesAfterCordovaKeyboard', e);
     }

src/js/common/utils/cordovaUtils.js

@@ -5,15 +5,13 @@
 /* eslint-disable func-names */
 /* eslint-disable indent */
 /* eslint-disable no-continue */
-/* eslint-disable no-empty */
 /* eslint-disable no-param-reassign */
 /* eslint-disable no-restricted-syntax */
 /* eslint-disable no-return-assign */
 /* eslint-disable no-shadow */
 /* eslint-disable object-shorthand */
 /* eslint-disable prefer-template */
 /* eslint-disable semi */
-/* eslint-disable semi */
 /* eslint-disable vars-on-top */
 
 import { isIOS } from '../cordovaUtils';
@@ -103,7 +101,10 @@ function init (converter, defaultAttributes) {
         if (name === found) {
           break
         }
-      } catch (e) {}
+        // eslint-disable-next-line no-unused-vars
+      } catch (e) {
+        continue;
+      }
     }
 
     return name ? jar[name] : jar

src/js/common/utils/js-cookie/Cookies.js

@@ -1,7 +1,6 @@
 /* eslint-disable comma-dangle */
 /* eslint-disable func-names */
 /* eslint-disable no-param-reassign */
-/* eslint-disable no-var */
 /* eslint-disable object-shorthand */
 /* eslint-disable semi */
 
@@ -19,4 +18,4 @@ export default {
     );
   }
 }
-/* eslint-enable no-var */
+