Merge pull request #654 from kitspace/ke/search-url-delay

Delay search url updating to not add lots of half written queries

e2e/cypress/e2e/multiProject.spec.js

@@ -189,6 +189,8 @@ describe('Multi project page', () => {
     // Search for the project
     cy.get('[data-cy=search-field] > input').type(user.username)
 
+    cy.wait(1000)
+
     // Click on a multiproject project card
     cy.get('[data-cy=project-card]')
       .first()

frontend/src/components/SearchInput.tsx

@@ -27,13 +27,29 @@ const SearchInput = () => {
   }, [routerQuery])
 
   const debouncedSubmit = useRef(debounce(value => updateContextQuery(value), 100))
+  const debouncedUpdateUrl = useRef(
+    debounce(value => {
+      const path = value ? `/search?q=${encodeURIComponent(value)}` : '/'
+      replace(path, undefined, { shallow: true })
+    }, 500),
+  )
+
+  // Cancel on component dismount
+  useEffect(() => {
+    return () => {
+      // I don't understand what react / eslint wants us to do in this case:
+      // the ref doesn't actually change.
+      // eslint-disable-next-line react-hooks/exhaustive-deps
+      debouncedUpdateUrl.current.cancel()
+    }
+  }, [])
 
   const handleChange = (value: string) => {
     debouncedSubmit.current.cancel()
+    debouncedUpdateUrl.current.cancel()
     setInputQuery(value)
     debouncedSubmit.current(value)
-    const path = value ? `/search?q=${encodeURIComponent(value)}` : '/'
-    replace(path, undefined, { shallow: true })
+    debouncedUpdateUrl.current(value)
   }
 
   return (

processor/Dockerfile

@@ -30,5 +30,7 @@ RUN addgroup --gid 1000 node && \
     mkdir /data /gitea-data && \
     chown -R node /data /gitea-data
 
+RUN git config --global --add safe.directory '*'
+
 USER node
 CMD ["node", "dist/src/main.js"]

processor/src/queues.ts

@@ -218,6 +218,7 @@ async function sync(gitDir, checkoutDir) {
         try {
           await sh`git clone ${gitDir} ${checkoutDir}`
         } catch (err) {
+          log.error(err)
           if (err.stderr) {
             return
           }

scripts/importBoardsTxt.ts

@@ -299,7 +299,8 @@ async function generateGiteaAdminToken() {
   )
 
   if (!giteaAdminCommand.status.success) {
-    throw new Error('Failed to create gitea admin user')
+    console.error(giteaAdminCommand.output)
+    throw new Error("Failed to create gitea admin user")
   }
 
   const giteaAdminTokenCommand = await exec(
@@ -308,7 +309,8 @@ async function generateGiteaAdminToken() {
   )
 
   if (!giteaAdminTokenCommand.status.success) {
-    throw new Error('Failed to create gitea admin token')
+    console.error(giteaAdminTokenCommand.output)
+    throw new Error("Failed to create gitea admin token")
   }
 
   return giteaAdminTokenCommand.output.split('\n').at(-1)