React18 (#872)

Author: laszlocph

Diff for: pkg/dashboard/server/gitContents.go

@@ -41,11 +41,17 @@ func branches(w http.ResponseWriter, r *http.Request) {
 	gitRepoCache, _ := ctx.Value("gitRepoCache").(*nativeGit.RepoCache)
 
 	var refIter storer.ReferenceIter
+	var innerErr error
 	branches := []string{}
-	gitRepoCache.PerformAction(repoName, func(repo *git.Repository) error {
-		refIter, _ = repo.References()
-		return nil
+	err := gitRepoCache.PerformAction(repoName, func(repo *git.Repository) error {
+		refIter, innerErr = repo.References()
+		return innerErr
 	})
+	if err != nil {
+		logrus.Errorf("cannot get refs: %s", err)
+		http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
+		return
+	}
 	refIter.ForEach(func(r *plumbing.Reference) error {
 		if r.Name().IsRemote() {
 			branch := r.Name().Short()

Diff for: web/eslint.config.mjs

@@ -0,0 +1,11 @@
+import js from "@eslint/js";
+
+export default [
+  js.configs.recommended,
+  {
+    rules: {
+      "no-unused-vars": "warn",
+      "no-undef": "warn"
+    }
+  }
+];

Diff for: web/package-lock.json

@@ -3,30 +3,30 @@
   "version": "0.1.0",
   "private": true,
   "dependencies": {
-    "@headlessui/react": "^1.7.18",
+    "@headlessui/react": "^2.1.8",
     "@heroicons/react": "^2.1.1",
-    "@testing-library/jest-dom": "^5.17.0",
-    "@testing-library/react": "^11.2.6",
-    "@testing-library/user-event": "^12.8.3",
-    "@types/jest": "^26.0.23",
-    "@types/react": "^17.0.75",
-    "@types/react-dom": "^17.0.23",
+    "@testing-library/jest-dom": "^6.5.0",
+    "@testing-library/react": "^16.0.1",
+    "@testing-library/user-event": "^14.5.2",
+    "@types/jest": "^29.5.13",
+    "@types/react": "^18.3.9",
+    "@types/react-dom": "^18.3.0",
     "axios": "^1.7.7",
-    "date-fns": "^3.6.0",
-    "diff": "^5.2.0",
+    "date-fns": "^4.1.0",
+    "diff": "^7.0.0",
     "helm-react-ui": "github:gimlet-io/helm-react-ui#master",
     "immer": "^10.1.1",
     "js-yaml": "^4.1.0",
     "jsonpath": "^1.1.1",
     "nanoid": "^5.0.7",
-    "posthog-js": "^1.161.2",
-    "react": "^16.5.2",
+    "posthog-js": "^1.165.0",
+    "react": "^18.3.1",
     "react-confetti": "^6.1.0",
     "react-diff-viewer-continued": "^3.4.0",
-    "react-dom": "^16.5.2",
+    "react-dom": "^18.3.1",
     "react-emoji-render": "^2.0.1",
     "react-infinite-scroll-component": "^6.1.0",
-    "react-router-dom": "^5.3.4",
+    "react-router-dom": "^6.26.2",
     "redux": "^5.0.1",
     "remarkable": "^2.0.1",
     "typescript": "^5.6.2",
@@ -62,18 +62,17 @@
     "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
     "@tailwindcss/forms": "^0.5.9",
     "@tailwindcss/typography": "^0.5.15",
-    "@types/node": "^22.5.4",
+    "@types/node": "^22.7.0",
     "@vitejs/plugin-react": "^4.2.1",
     "@vitejs/plugin-react-swc": "^3.6.0",
-    "@vitest/coverage-v8": "^2.0.5",
+    "@vitest/coverage-v8": "^2.1.1",
     "autoprefixer": "^10.4.7",
-    "eslint": "^8.57.0",
-    "eslint-config-react-app": "^7.0.1",
-    "postcss": "^8.4.31",
-    "tailwindcss": "^3.4.1",
-    "vite": "^5.4.3",
+    "eslint": "^9.11.1",
+    "postcss": "^8.4.47",
+    "tailwindcss": "^3.4.13",
+    "vite": "^5.4.8",
     "vite-plugin-eslint": "^1.8.1",
-    "vitest": "^2.0.5"
+    "vitest": "^2.1.1"
   },
   "proxy": "http://127.0.0.1:9000"
 }

Diff for: web/package.json

@@ -1,37 +1,38 @@
-import { Component } from 'react';
+import { useEffect } from 'react';
 import {
   ACTION_TYPE_ENVS,
   ACTION_FLUX_EVENTS_RECEIVED,
   ACTION_TYPE_USERS,
   ACTION_TYPE_APPLICATION,
   ACTION_TYPE_ALERTS,
 } from "./redux/redux";
+import { useLocation } from 'react-router-dom';
 
-export default class APIBackend extends Component {
+export default function APIBackend(props) {
+  const { gimletClient, store } = props
+  const location = useLocation()
 
-  componentDidMount() {
-    if (this.props.location.pathname.startsWith('/login')) {
+  useEffect(() => {
+    if (location.pathname.startsWith('/login')) {
       return;
     }
 
-    this.props.gimletClient.getUsers()
-      .then(data => this.props.store.dispatch({ type: ACTION_TYPE_USERS, payload: data }), () => {/* Generic error handler deals with it */
+    gimletClient.getUsers()
+      .then(data => store.dispatch({ type: ACTION_TYPE_USERS, payload: data }), () => {/* Generic error handler deals with it */
       });
-    this.props.gimletClient.getApp()
-      .then(data => this.props.store.dispatch({ type: ACTION_TYPE_APPLICATION, payload: data }), () => {/* Generic error handler deals with it */
+    gimletClient.getApp()
+      .then(data => store.dispatch({ type: ACTION_TYPE_APPLICATION, payload: data }), () => {/* Generic error handler deals with it */
       });
-    this.props.gimletClient.getEnvs()
-      .then(data => this.props.store.dispatch({ type: ACTION_TYPE_ENVS, payload: data }), () => {/* Generic error handler deals with it */
+    gimletClient.getEnvs()
+      .then(data => store.dispatch({ type: ACTION_TYPE_ENVS, payload: data }), () => {/* Generic error handler deals with it */
       });
-    this.props.gimletClient.getFluxEvents()
-      .then(data => this.props.store.dispatch({ type: ACTION_FLUX_EVENTS_RECEIVED, payload: data }), () => {/* Generic error handler deals with it */
+    gimletClient.getFluxEvents()
+      .then(data => store.dispatch({ type: ACTION_FLUX_EVENTS_RECEIVED, payload: data }), () => {/* Generic error handler deals with it */
       });
-    this.props.gimletClient.getAlerts()
-      .then(data => this.props.store.dispatch({ type: ACTION_TYPE_ALERTS, payload: data }), () => {/* Generic error handler deals with it */
+    gimletClient.getAlerts()
+      .then(data => store.dispatch({ type: ACTION_TYPE_ALERTS, payload: data }), () => {/* Generic error handler deals with it */
       });
-  }
+  })
 
-  render() {
-    return null;
-  }
+  return null;
 }