types for all auth related things (#135)

src/Components/Auth/LoginPage.tsx

@@ -1,8 +1,3 @@
-/* eslint-disable @typescript-eslint/no-misused-promises */
-/* eslint-disable @typescript-eslint/restrict-template-expressions */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-floating-promises */
 import { useEffect, useState } from 'react'
 import { Link, useNavigate } from 'react-router-dom'
 import { toast } from 'react-toastify'
@@ -28,14 +23,14 @@ export function LoginPage() {
       success: {
         render({ data }) {
           navigate('/')
-          return `Hi ${data?.first_name}`
+          return `Hi ${data?.first_name ? data.first_name : 'Traveler'}`
         },
         // other options
         icon: '✌️',
       },
       error: {
         render({ data }) {
-          return `${data}`
+          return `${data as string}`
         },
         autoClose: 10000,
       },
@@ -44,9 +39,10 @@ export function LoginPage() {
   }
 
   useEffect(() => {
-    const keyDownHandler = (event) => {
+    const keyDownHandler = (event: KeyboardEvent) => {
       if (event.key === 'Enter') {
         event.preventDefault()
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         onLogin()
       }
     }
@@ -86,6 +82,7 @@ export function LoginPage() {
               ? 'tw-btn tw-btn-disabled tw-btn-block tw-btn-primary'
               : 'tw-btn tw-btn-primary tw-btn-block'
           }
+          // eslint-disable-next-line @typescript-eslint/no-misused-promises
           onClick={() => onLogin()}
         >
           {loading ? <span className='tw-loading tw-loading-spinner'></span> : 'Login'}

src/Components/Auth/RequestPasswordPage.tsx

@@ -1,6 +1,3 @@
-/* eslint-disable @typescript-eslint/restrict-template-expressions */
-/* eslint-disable @typescript-eslint/no-misused-promises */
-/* eslint-disable @typescript-eslint/no-unsafe-argument */
 import { useState } from 'react'
 import { useNavigate } from 'react-router-dom'
 import { toast } from 'react-toastify'
@@ -12,8 +9,7 @@ import { useAuth } from './useAuth'
 /**
  * @category Auth
  */
-// eslint-disable-next-line react/prop-types
-export function RequestPasswordPage({ resetUrl }) {
+export function RequestPasswordPage({ resetUrl }: { resetUrl: string }) {
   const [email, setEmail] = useState<string>('')
 
   const { requestPasswordReset, loading } = useAuth()
@@ -32,7 +28,7 @@ export function RequestPasswordPage({ resetUrl }) {
       },
       error: {
         render({ data }) {
-          return `${data}`
+          return `${data as string}`
         },
       },
       pending: 'sending email ...',
@@ -56,6 +52,7 @@ export function RequestPasswordPage({ resetUrl }) {
               ? 'tw-btn tw-btn-disabled tw-btn-block tw-btn-primary'
               : 'tw-btn tw-btn-primary tw-btn-block'
           }
+          // eslint-disable-next-line @typescript-eslint/no-misused-promises
           onClick={() => onReset()}
         >
           {loading ? <span className='tw-loading tw-loading-spinner'></span> : 'Send'}

src/Components/Auth/SetNewPasswordPage.tsx

@@ -1,5 +1,3 @@
-/* eslint-disable @typescript-eslint/no-misused-promises */
-/* eslint-disable @typescript-eslint/restrict-template-expressions */
 import { useState } from 'react'
 import { useNavigate } from 'react-router-dom'
 import { toast } from 'react-toastify'
@@ -20,8 +18,6 @@ export function SetNewPasswordPage() {
 
   const onReset = async () => {
     const token = window.location.search.split('token=')[1]
-    // eslint-disable-next-line no-console
-    console.log(token)
 
     await toast.promise(passwordReset(token, password), {
       success: {
@@ -32,7 +28,7 @@ export function SetNewPasswordPage() {
       },
       error: {
         render({ data }) {
-          return `${data}`
+          return `${data as string}`
         },
       },
       pending: 'setting password ...',
@@ -55,6 +51,7 @@ export function SetNewPasswordPage() {
               ? 'tw-btn tw-btn-disabled tw-btn-block tw-btn-primary'
               : 'tw-btn tw-btn-primary tw-btn-block'
           }
+          // eslint-disable-next-line @typescript-eslint/no-misused-promises
           onClick={() => onReset()}
         >
           {loading ? <span className='tw-loading tw-loading-spinner'></span> : 'Set'}

src/Components/Auth/SignupPage.tsx

@@ -1,8 +1,3 @@
-/* eslint-disable @typescript-eslint/restrict-template-expressions */
-/* eslint-disable @typescript-eslint/no-floating-promises */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-misused-promises */
 import { useEffect, useState } from 'react'
 import { useNavigate } from 'react-router-dom'
 import { toast } from 'react-toastify'
@@ -30,14 +25,14 @@ export function SignupPage() {
       success: {
         render({ data }) {
           navigate('/')
-          return `Hi ${data?.first_name}`
+          return `Hi ${data?.first_name ? data.first_name : 'Traveler'}`
         },
         // other options
         icon: '✌️',
       },
       error: {
         render({ data }) {
-          return `${data}`
+          return `${data as string}`
         },
         autoClose: 10000,
       },
@@ -46,9 +41,10 @@ export function SignupPage() {
   }
 
   useEffect(() => {
-    const keyDownHandler = (event) => {
+    const keyDownHandler = (event: KeyboardEvent) => {
       if (event.key === 'Enter') {
         event.preventDefault()
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         onRegister()
       }
     }
@@ -88,6 +84,7 @@ export function SignupPage() {
               ? 'tw-btn tw-btn-disabled tw-btn-block tw-btn-primary'
               : 'tw-btn tw-btn-primary tw-btn-block'
           }
+          // eslint-disable-next-line @typescript-eslint/no-misused-promises
           onClick={() => onRegister()}
         >
           {loading ? <span className='tw-loading tw-loading-spinner'></span> : 'Sign Up'}

src/Components/Auth/useAuth.tsx

@@ -1,8 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unused-vars */
-/* eslint-disable @typescript-eslint/no-unsafe-return */
-/* eslint-disable @typescript-eslint/no-unsafe-argument */
-/* eslint-disable @typescript-eslint/no-floating-promises */
-/* eslint-disable @typescript-eslint/no-explicit-any */
 import { createContext, useState, useContext, useEffect } from 'react'
 
 import type { UserApi } from '#types/UserApi'
@@ -28,11 +23,11 @@ interface AuthContextProps {
   login: (credentials: AuthCredentials) => Promise<UserItem | undefined>
   register: (credentials: AuthCredentials, userName: string) => Promise<UserItem | undefined>
   loading: boolean
-  logout: () => Promise<any>
-  updateUser: (user: UserItem) => any
-  token: string | null
-  requestPasswordReset: (email: string, reset_url: string) => Promise<any>
-  passwordReset: (token: string, new_password: string) => Promise<any>
+  logout: () => Promise<void>
+  updateUser: (user: UserItem) => Promise<UserItem>
+  token: string | undefined
+  requestPasswordReset: (email: string, reset_url: string) => Promise<void>
+  passwordReset: (token: string, new_password: string) => Promise<void>
 }
 
 const AuthContext = createContext<AuthContextProps>({
@@ -53,12 +48,13 @@ const AuthContext = createContext<AuthContextProps>({
  */
 export const AuthProvider = ({ userApi, children }: AuthProviderProps) => {
   const [user, setUser] = useState<UserItem | null>(null)
-  const [token, setToken] = useState<string | null>(null)
+  const [token, setToken] = useState<string>()
   const [loading, setLoading] = useState<boolean>(false)
   const isAuthenticated = !!user
 
   useEffect(() => {
     setLoading(true)
+    // eslint-disable-next-line @typescript-eslint/no-floating-promises
     loadUser()
     setLoading(false)
     // eslint-disable-next-line react-hooks/exhaustive-deps
@@ -84,10 +80,10 @@ export const AuthProvider = ({ userApi, children }: AuthProviderProps) => {
   const login = async (credentials: AuthCredentials): Promise<UserItem | undefined> => {
     setLoading(true)
     try {
-      const res = await userApi.login(credentials.email, credentials.password)
-      setToken(res?.access_token)
+      const user = await userApi.login(credentials.email, credentials.password)
+      setToken(user?.access_token)
       return await loadUser()
-    } catch (error: any) {
+    } catch (error) {
       setLoading(false)
       throw error
     }
@@ -95,13 +91,13 @@ export const AuthProvider = ({ userApi, children }: AuthProviderProps) => {
 
   const register = async (
     credentials: AuthCredentials,
-    userName,
+    userName: string,
   ): Promise<UserItem | undefined> => {
     setLoading(true)
     try {
       /* const res = */ await userApi.register(credentials.email, credentials.password, userName)
       return await login(credentials)
-    } catch (error: any) {
+    } catch (error) {
       setLoading(false)
       throw error
     }
@@ -111,7 +107,7 @@ export const AuthProvider = ({ userApi, children }: AuthProviderProps) => {
     try {
       await userApi.logout()
       setUser(null)
-    } catch (error: any) {
+    } catch (error) {
       setLoading(false)
       throw error
     }
@@ -119,37 +115,35 @@ export const AuthProvider = ({ userApi, children }: AuthProviderProps) => {
 
   const updateUser = async (user: UserItem) => {
     setLoading(true)
-    const { id, ...userRest } = user
-
     try {
-      const res = await userApi.updateUser(userRest)
-      setUser(res as any)
-      loadUser()
+      const updatedUser = await userApi.updateUser(user)
+      setUser(updatedUser)
+      await loadUser()
       setLoading(false)
-      return res as any
-    } catch (error: any) {
+      return updatedUser
+    } catch (error) {
       setLoading(false)
       throw error
     }
   }
 
-  const requestPasswordReset = async (email: string, resetUrl?: string): Promise<any> => {
+  const requestPasswordReset = async (email: string, resetUrl?: string): Promise<void> => {
     setLoading(true)
     try {
       await userApi.requestPasswordReset(email, resetUrl)
       return setLoading(false)
-    } catch (error: any) {
+    } catch (error) {
       setLoading(false)
       throw error
     }
   }
 
-  const passwordReset = async (token: string, newPassword: string): Promise<any> => {
+  const passwordReset = async (token: string, newPassword: string): Promise<void> => {
     setLoading(true)
     try {
       await userApi.passwordReset(token, newPassword)
       return setLoading(false)
-    } catch (error: any) {
+    } catch (error) {
       setLoading(false)
       throw error
     }

src/Components/Profile/UserSettings.tsx

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unsafe-argument */
 /* eslint-disable @typescript-eslint/restrict-template-expressions */
 import { useEffect, useState } from 'react'
 import { useNavigate } from 'react-router-dom'

src/types/UserApi.d.ts

@@ -8,8 +8,8 @@ export interface UserApi {
   login(email: string, password: string): Promise<UserItem | undefined>
   logout(): Promise<void>
   getUser(): Promise<UserItem>
-  getToken(): Promise<string | null>
-  updateUser(user: UserItem): Promise<void>
+  getToken(): Promise<string | undefined>
+  updateUser(user: UserItem): Promise<UserItem>
   requestPasswordReset(email: string, reset_url?: string)
   passwordReset(token: string, new_password: string)
 }

src/types/UserItem.d.ts

@@ -11,6 +11,7 @@ export interface UserItem {
   password?: string
   profile?: Profile
   first_name?: string
+  access_token?: string
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any
 }