[Issue HHS#2037]Add no console log to eslint rules (#182)

Fixes HHS#2037 > Added new rule to eslint to prevent leaving behind console log statements > normal expected rule in eslint configurations
navapbc · Sep 18, 2024 · 7a0ce80 · 7a0ce80
1 parent 766a7e6
commit 7a0ce80
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/frontend/.eslintrc.js b/frontend/.eslintrc.js
@@ -51,6 +51,8 @@ module.exports = {
         "@typescript-eslint/no-explicit-any": "error",
         // Just warn since playwright tests may not use screen the way jest would
         "testing-library/prefer-screen-queries": "warn",
+        // Prevent unnecessary console statements
+        "no-console": ["error", { allow: ["warn", "error"] }],
       },
     },
   ],

diff --git a/frontend/src/app/api/BaseApi.ts b/frontend/src/app/api/BaseApi.ts
@@ -219,7 +219,7 @@ const throwError = (
   searchInputs?: QueryParamData,
   firstError?: APIResponseError,
 ) => {
-  console.log("Throwing error: ", message, status_code, searchInputs);
+  console.error("Throwing error: ", message, status_code, searchInputs);
 
   // Include just firstError for now, we can expand this
   // If we need ValidationErrors to be more expanded