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

Fixes #2037

> Added new rule to eslint to prevent leaving behind console log
statements

> normal expected rule in eslint configurations

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"] }],
       },
     },
   ],

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