review comments fix

.eslintrc.json

@@ -37,7 +37,7 @@
     "ecmaVersion": "latest",
     "sourceType": "module"
   },
-  "plugins": ["@typescript-eslint", "tailwindcss", "react", "react-hooks", "import", "prettier", "unused-imports"],
+  "plugins": ["@typescript-eslint", "tailwindcss", "react", "react-hooks", "import", "prettier"],
   "rules": {
     "tailwindcss/no-contradicting-classname": "error",
     "import/no-unresolved": "off",
@@ -47,15 +47,11 @@
     "jsx-a11y/click-events-have-key-events": "warn",
     "jsx-a11y/no-autofocus": "off",
 
-    "@typescript-eslint/no-unused-vars": "off",
-    "unused-imports/no-unused-imports": "error",
-    "unused-imports/no-unused-vars": [
-      "warn",
+    "@typescript-eslint/no-unused-vars": [
+      "error",
       {
-        "vars": "all",
-        "args": "after-used",
-        "varsIgnorePattern": "^_",
         "argsIgnorePattern": "^_",
+        "varsIgnorePattern": "^_",
         "caughtErrorsIgnorePattern": "^_"
       }
     ],

.vscode/settings.json

@@ -9,10 +9,5 @@
   "[ignore]": {
     "editor.defaultFormatter": "foxundermoon.shell-format"
   },
-  "prettier.trailingComma": "none",
-  "editor.codeActionsOnSave": {
-    "source.fixAll": "always",
-    "source.fixAll.eslint": "always"
-  },
-  "eslint.validate": ["javascript", "javascriptreact", "typescript", "typescriptreact"]
+  "prettier.trailingComma": "none"
 }

apps/design-system/src/pages/view-preview/profile-settings-view-wrapper.tsx

@@ -1,14 +1,14 @@
 import { FC, HTMLAttributes, PropsWithChildren } from 'react'
 import { Route, Routes } from 'react-router-dom'
 
-import { useTranslationsStore } from '@utils/viewUtils'
+import { useTranslationStore } from '@utils/viewUtils'
 
 import { ProfileSettingsLayout } from '@harnessio/ui/views'
 
 export const ProfileSettingsViewWrapper: FC<PropsWithChildren<HTMLAttributes<HTMLElement>>> = ({ children }) => {
   return (
     <Routes>
-      <Route path="*" element={<ProfileSettingsLayout useTranslationStore={useTranslationsStore} />}>
+      <Route path="*" element={<ProfileSettingsLayout useTranslationStore={useTranslationStore} />}>
         <Route path="*" element={children} />
       </Route>
     </Routes>

apps/design-system/src/subjects/views/labels/labels-form.tsx

@@ -1,13 +1,13 @@
 import { LabelsListStore } from '@subjects/stores/labels-store.tsx'
-import { useTranslationsStore } from '@utils/viewUtils.ts'
+import { useTranslationStore } from '@utils/viewUtils.ts'
 
 import { LabelFormPage } from '@harnessio/ui/views'
 
 export const LabelsForm = () => {
   return (
     <LabelFormPage
       useLabelsStore={LabelsListStore.useLabelsStore}
-      useTranslationStore={useTranslationsStore}
+      useTranslationStore={useTranslationStore}
       isLoading={false}
       onSubmit={() => {}}
       onFormCancel={() => {}}

apps/design-system/src/subjects/views/profile-settings-keys/index.tsx

@@ -1,6 +1,6 @@
 import { useState } from 'react'
 
-import { noop, useTranslationsStore } from '@utils/viewUtils'
+import { noop, useTranslationStore } from '@utils/viewUtils'
 
 import { DeleteAlertDialog } from '@harnessio/ui/components'
 import {
@@ -24,7 +24,7 @@ export const ProfileSettingsKeysView = () => {
         openSshKeyDialog={() => setIsKeysDialogOpen(true)}
         openAlertDeleteDialog={() => setIsDeleteDialogOpen(true)}
         error={null}
-        useTranslationStore={useTranslationsStore}
+        useTranslationStore={useTranslationStore}
         isLoadingTokenList={false}
         isLoadingKeysList={false}
       />
@@ -34,23 +34,23 @@ export const ProfileSettingsKeysView = () => {
         handleCreateToken={noop}
         error={null}
         isLoading={false}
-        useTranslationStore={useTranslationsStore}
+        useTranslationStore={useTranslationStore}
         useProfileSettingsStore={mockProfileSettingsStore}
       />
       <ProfileSettingsKeysCreateDialog
         open={isKeysDialogOpen}
         onClose={() => setIsKeysDialogOpen(false)}
         handleCreateSshKey={noop}
         error={null}
-        useTranslationStore={useTranslationsStore}
+        useTranslationStore={useTranslationStore}
       />
       <DeleteAlertDialog
         open={isDeleteDialogOpen}
         onClose={() => setIsDeleteDialogOpen(false)}
         deleteFn={noop}
         error={null}
         isLoading={false}
-        useTranslationStore={useTranslationsStore}
+        useTranslationStore={useTranslationStore}
       />
     </>
   )

apps/design-system/src/subjects/views/profile-settings/index.tsx

@@ -1,6 +1,6 @@
 import { useEffect, useState } from 'react'
 
-import { noop, useTranslationsStore } from '@utils/viewUtils'
+import { noop, useTranslationStore } from '@utils/viewUtils'
 
 import { SettingsAccountGeneralPage } from '@harnessio/ui/views'
 
@@ -16,7 +16,7 @@ export const ProfileSettingsView = () => {
   return (
     <SettingsAccountGeneralPage
       useProfileSettingsStore={mockProfileSettingsStore}
-      useTranslationStore={useTranslationsStore}
+      useTranslationStore={useTranslationStore}
       isLoadingUser={isLoadingUser}
       isUpdatingUser={false}
       isUpdatingPassword={false}

apps/design-system/src/subjects/views/repo-create/index.tsx

@@ -1,4 +1,4 @@
-import { noop, useTranslationsStore } from '@utils/viewUtils'
+import { noop, useTranslationStore } from '@utils/viewUtils'
 
 import { RepoCreatePage } from '@harnessio/ui/views'
 
@@ -15,7 +15,7 @@ export const CreateRepoView = () => {
       isSuccess={false}
       gitIgnoreOptions={gitIgnoreOptions}
       licenseOptions={licenseOptions}
-      useTranslationStore={useTranslationsStore}
+      useTranslationStore={useTranslationStore}
     />
   )
 }

apps/design-system/src/subjects/views/repo-import/index.tsx

@@ -1,4 +1,4 @@
-import { noop, useTranslationsStore } from '@utils/viewUtils'
+import { noop, useTranslationStore } from '@utils/viewUtils'
 
 import { RepoImportPage } from '@harnessio/ui/views'
 
@@ -10,7 +10,7 @@ export const ImportRepoView = () => {
         onFormCancel={noop}
         isLoading={false}
         apiErrorsValue={undefined}
-        useTranslationStore={useTranslationsStore}
+        useTranslationStore={useTranslationStore}
       />
     </>
   )

apps/gitness/src/framework/context/AppContext.tsx

@@ -18,6 +18,7 @@ import usePageTitle from '../hooks/usePageTitle'
 
 interface AppContextType {
   spaces: TypesSpace[]
+  isSpacesLoading: boolean
   setSpaces: (value: TypesSpace[]) => void
   addSpaces: (newSpaces: TypesSpace[]) => void
   currentUser?: TypesUser
@@ -34,6 +35,7 @@ interface AppContextType {
 
 const AppContext = createContext<AppContextType>({
   spaces: [],
+  isSpacesLoading: false,
   setSpaces: noop,
   addSpaces: noop,
   currentUser: undefined,
@@ -48,6 +50,7 @@ const AppContext = createContext<AppContextType>({
 export const AppProvider: FC<{ children: ReactNode }> = ({ children }) => {
   usePageTitle()
   const [spaces, setSpaces] = useState<TypesSpace[]>([])
+  const [isSpacesLoading, setSpacesIsLoading] = useState<boolean>(false)
   const [currentUser, setCurrentUser] = useLocalStorage<TypesUser>('currentUser', {})
   const [isLoadingUser, setIsLoadingUser] = useState(false)
   const [isUpdatingUser, setIsUpdatingUser] = useState(false)
@@ -91,6 +94,7 @@ export const AppProvider: FC<{ children: ReactNode }> = ({ children }) => {
   }
 
   useEffect(() => {
+    setSpacesIsLoading(true)
     Promise.allSettled([
       membershipSpaces({
         queryParams: { page: 1, limit: 100, sort: 'identifier', order: 'asc' }
@@ -102,6 +106,7 @@ export const AppProvider: FC<{ children: ReactNode }> = ({ children }) => {
 
         if (membershipResult.status === 'fulfilled') {
           setSpaces(membershipResult.value.body.filter(item => item?.space).map(item => item.space as TypesSpace))
+          setSpacesIsLoading(false)
         }
       })
       .catch(() => {
@@ -125,7 +130,8 @@ export const AppProvider: FC<{ children: ReactNode }> = ({ children }) => {
         updateUserProfile,
         isLoadingUser,
         isUpdatingUser,
-        updateUserError
+        updateUserError,
+        isSpacesLoading
       }}
     >
       {children}

apps/gitness/src/pages-v2/project/project-general-settings-container.tsx

@@ -18,21 +18,15 @@ import { useSpaceStore } from './stores/spaces-store'
 
 export const ProjectGeneralSettingsPageContainer = () => {
   const spaceURL = useGetSpaceURLParam()
-  const { spaces } = useAppContext()
-  const { setSpace } = useSpaceStore()
+  const { spaces, isSpacesLoading } = useAppContext()
+  const { setSpace, setIsLoading } = useSpaceStore()
   const space = spaces.find((space: TypesSpace) => space?.identifier === spaceURL)
   const [openDeleteDialog, setOpenDeleteDialog] = useState(false)
   const [updateError, setUpdateError] = useState<string | null>(null)
   const [deleteError, setDeleteError] = useState<{ type: string; message: string } | null>(null)
 
-  useEffect(() => {
-    setSpace(space || null)
-  }, [space, setSpace])
-
   const updateDescription = useUpdateSpaceMutation(
-    {
-      space_ref: space?.path
-    },
+    { space_ref: space?.path },
     {
       onSuccess: ({ body: data }) => {
         if (space) {
@@ -48,8 +42,8 @@ export const ProjectGeneralSettingsPageContainer = () => {
     }
   )
 
-  const handleFormSubmit = (body: ProjectSettingsGeneralFields) => {
-    updateDescription.mutate({ space_ref: space?.path, body })
+  const handleFormSubmit = (formdata: ProjectSettingsGeneralFields) => {
+    updateDescription.mutate({ space_ref: space?.path, body: { description: formdata?.description } })
   }
 
   // delete API call here
@@ -59,7 +53,7 @@ export const ProjectGeneralSettingsPageContainer = () => {
       onSuccess: ({ body: data }) => {
         if (data) {
           setDeleteError(null)
-          window.location.href = '/'
+          redirect('/')
         }
       },
       onError: (error: DeleteSpaceErrorResponse) => {
@@ -75,12 +69,20 @@ export const ProjectGeneralSettingsPageContainer = () => {
       {
         onSuccess: () => {
           setDeleteError(null)
-          window.location.href = '/'
+          redirect('/')
         }
       }
     )
   }
 
+  useEffect(() => {
+    setIsLoading(isSpacesLoading)
+  }, [isSpacesLoading, setIsLoading])
+
+  useEffect(() => {
+    setSpace(space || null)
+  }, [space, setSpace])
+
   return (
     <>
       <ProjectSettingsGeneralPage

apps/gitness/src/pages-v2/project/stores/spaces-store.ts

@@ -4,5 +4,7 @@ import { ISpaceStore } from '@harnessio/ui/views'
 
 export const useSpaceStore = create<ISpaceStore>(set => ({
   space: null,
-  setSpace: space => set({ space })
+  isLoading: false,
+  setSpace: space => set({ space }),
+  setIsLoading: isLoading => set({ isLoading })
 }))

apps/gitness/src/routes.tsx

@@ -16,7 +16,6 @@ import PipelineEditPage from './pages-v2/pipeline/pipeline-edit/pipeline-edit'
 import ProjectPipelineListPage from './pages-v2/pipeline/project-pipeline-list-page'
 import { SettingsProfileGeneralPage } from './pages-v2/profile-settings/profile-settings-general-container'
 import { SettingsProfileKeysPage } from './pages-v2/profile-settings/profile-settings-keys-container'
-import { ProfileSettingsLayout } from './pages-v2/profile-settings/profile-settings-layout'
 import { ProfileSettingsThemePage } from './pages-v2/profile-settings/profile-settings-theme-page'
 import { ProjectLabelFormContainer } from './pages-v2/project/labels/project-label-form-container'
 import { ProjectLabelsList } from './pages-v2/project/labels/project-labels-list-container'

package.json

@@ -38,12 +38,11 @@
     "eslint-plugin-react": "^7.37.2",
     "eslint-plugin-react-hooks": "^5.0.0",
     "eslint-plugin-tailwindcss": "^3.17.5",
-    "eslint-plugin-unused-imports": "3.2.0",
     "husky": "^9.1.4",
     "lint-staged": "^15.2.9",
     "prettier": "^3.3.3",
     "prettier-plugin-tailwindcss": "^0.6.8",
     "rimraf": "^6.0.1",
     "typescript-eslint": "^8.14.0"
   }
-}
+}