Merge pull request #2002 from automatisch/aut-1255

feat: expose installationCompleted in GET /v1/automatisch/info

packages/backend/src/controllers/api/v1/automatisch/info.js

@@ -1,13 +1,17 @@
 import appConfig from '../../../../config/app.js';
 import { hasValidLicense } from '../../../../helpers/license.ee.js';
 import { renderObject } from '../../../../helpers/renderer.js';
+import Config from '../../../../models/config.js';
 
 export default async (request, response) => {
+  const installationCompleted = await Config.isInstallationCompleted();
+
   const info = {
-    isCloud: appConfig.isCloud,
-    isMation: appConfig.isMation,
-    isEnterprise: await hasValidLicense(),
     docsUrl: appConfig.docsUrl,
+    installationCompleted,
+    isCloud: appConfig.isCloud,
+    isEnterprise: await hasValidLicense(),
+    isMation: appConfig.isMation,
   };
 
   renderObject(response, info);

packages/backend/src/controllers/api/v1/automatisch/info.test.js

@@ -1,12 +1,14 @@
 import { vi, expect, describe, it } from 'vitest';
 import request from 'supertest';
 import appConfig from '../../../../config/app.js';
+import Config from '../../../../models/config.js';
 import app from '../../../../app.js';
 import infoMock from '../../../../../test/mocks/rest/api/v1/automatisch/info.js';
 import * as license from '../../../../helpers/license.ee.js';
 
 describe('GET /api/v1/automatisch/info', () => {
   it('should return Automatisch info', async () => {
+    vi.spyOn(Config, 'isInstallationCompleted').mockResolvedValue(true);
     vi.spyOn(appConfig, 'isCloud', 'get').mockReturnValue(false);
     vi.spyOn(appConfig, 'isMation', 'get').mockReturnValue(false);
     vi.spyOn(license, 'hasValidLicense').mockResolvedValue(true);

packages/backend/test/mocks/rest/api/v1/automatisch/info.js

@@ -1,10 +1,11 @@
 const infoMock = () => {
   return {
     data: {
-      isCloud: false,
-      isMation: false,
-      isEnterprise: true,
       docsUrl: 'https://automatisch.io/docs',
+      installationCompleted: true,
+      isCloud: false,
+      isEnterprise: true,
+      isMation: false,
     },
     meta: {
       count: 1,

packages/web/src/components/InstallationForm/index.jsx

@@ -44,7 +44,7 @@ function InstallationForm() {
 
   const handleOnRedirect = () => {
     queryClient.invalidateQueries({
-      queryKey: ['automatisch', 'config'],
+      queryKey: ['automatisch', 'info'],
     });
   };
 

packages/web/src/routes.jsx

@@ -29,14 +29,16 @@ import adminSettingsRoutes from './adminSettingsRoutes';
 import Notifications from 'pages/Notifications';
 import useAutomatischConfig from 'hooks/useAutomatischConfig';
 import useAuthentication from 'hooks/useAuthentication';
+import useAutomatischInfo from 'hooks/useAutomatischInfo';
 import Installation from 'pages/Installation';
 
 function Routes() {
-  const { data: configData, isSuccess } = useAutomatischConfig();
+  const { data: automatischInfo, isSuccess } = useAutomatischInfo();
+  const { data: configData } = useAutomatischConfig();
   const { isAuthenticated } = useAuthentication();
   const config = configData?.data;
 
-  const installed = isSuccess ? config?.['installation.completed'] === true : true;
+  const installed = isSuccess ? automatischInfo.data.installationCompleted : true;
   const navigate = useNavigate();
 
   useEffect(() => {