feat: update typescript eslint and make changes accordingly (#534)

Author: yoannfleurydev

package.json

@@ -125,7 +125,7 @@
     "@types/nodemailer": "6.4.15",
     "@types/react": "18.3.3",
     "@types/react-dom": "18.3.0",
-    "@typescript-eslint/eslint-plugin": "7.16.0",
+    "@typescript-eslint/eslint-plugin": "8.8.0",
     "@vitejs/plugin-react": "4.3.1",
     "@vitest/ui": "2.0.1",
     "cross-env": "7.0.3",

pnpm-lock.yaml

@@ -44,7 +44,12 @@ export const useDayPickerPopperManagement = (
   });
 
   const togglePopper = (withFocus = false) => {
-    isPopperOpen ? closePopper() : openPopper(withFocus);
+    if (isPopperOpen) {
+      closePopper();
+      return;
+    }
+
+    openPopper(withFocus);
   };
 
   // Si on click en dehors du calendrier, on le ferme

src/components/DayPicker/hooks/useDayPickerPopperManagement.ts

@@ -26,7 +26,7 @@ type FormFieldErrorProps<
   children?: (params: { error?: FieldError }) => ReactNode;
 } & (
     | Required<Pick<ControllerProps<TFieldValues, TName>, 'control' | 'name'>>
-    // eslint-disable-next-line @typescript-eslint/ban-types
+    // eslint-disable-next-line @typescript-eslint/no-empty-object-type
     | {}
   );
 

src/components/Form/FormFieldError.tsx

@@ -1,4 +1,4 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
+/* eslint-disable @typescript-eslint/no-require-imports */
 const template = require('./svgr.template');
 
 module.exports = {

src/components/Icons/svgr.config.js

@@ -1,7 +1,7 @@
 import { forwardRef } from 'react';
 
 // https://www.totaltypescript.com/forwardref-with-generic-components
-// eslint-disable-next-line @typescript-eslint/ban-types
+// eslint-disable-next-line @typescript-eslint/no-empty-object-type
 export function fixedForwardRef<T, P = {}>(
   render: (props: P, ref: React.Ref<T>) => React.ReactNode
 ): (props: P & React.RefAttributes<T>) => React.ReactNode {

src/lib/utils.ts

@@ -143,7 +143,7 @@ export async function validateCode({
           },
         },
       });
-    } catch (e) {
+    } catch {
       ctx.logger.error('Failed to update token attempts');
     }
 
@@ -168,7 +168,7 @@ export async function deleteUsedCode({
     await ctx.db.verificationToken.delete({
       where: { token },
     });
-  } catch (e) {
+  } catch {
     ctx.logger.warn('Failed to delete the used token');
   }
 }

src/server/config/auth.ts

@@ -174,7 +174,7 @@ export const authRouter = createTRPCRouter({
             lastLoginAt: new Date(),
           },
         });
-      } catch (e) {
+      } catch {
         ctx.logger.warn('Failed to update the user, probably not enabled');
         throw new TRPCError({
           code: 'UNAUTHORIZED',
@@ -370,7 +370,7 @@ export const authRouter = createTRPCRouter({
             accountStatus: 'ENABLED',
           },
         });
-      } catch (e) {
+      } catch {
         ctx.logger.warn('Failed to update the user, probably already verified');
         throw new TRPCError({
           code: 'UNAUTHORIZED',