@truedat/df 8.6.7 → 8.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/df",
-  "version": "8.6.7",
+  "version": "8.7.1",
   "description": "Truedat Web Data Quality Module",
   "sideEffects": false,
   "module": "src/index.js",
@@ -51,14 +51,14 @@
     "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/react": "^16.3.0",
     "@testing-library/user-event": "^14.6.1",
-    "@truedat/test": "8.6.7",
+    "@truedat/test": "8.7.1",
     "identity-obj-proxy": "^3.0.0",
     "jest": "^29.7.0",
     "redux-saga-test-plan": "^4.0.6"
   },
   "dependencies": {
     "@apollo/client": "^3.13.8",
-    "@truedat/core": "8.6.7",
+    "@truedat/core": "8.7.1",
     "axios": "^1.15.0",
     "graphql": "^16.11.0",
     "is-hotkey": "^0.2.0",
@@ -87,5 +87,5 @@
     "semantic-ui-react": "^3.0.0-beta.2",
     "swr": "^2.3.3"
   },
-  "gitHead": "17b10e045d4b90147c0f98ed5e781c20804024ba"
+  "gitHead": "f7605a6ea3f78839b42bb7d8339eec125ba49acb"
 }

package/src/components/DynamicFormViewer.js

@@ -31,9 +31,9 @@ export const DynamicFormViewer = ({
       name={
         name
           ? formatMessage({
-            id: `templates.groups.${name}`,
-            defaultMessage: name,
-          })
+              id: `templates.groups.${name}`,
+              defaultMessage: name,
+            })
           : null
       }
       fields={fields}

package/src/components/DynamicFormWithTranslations.js

@@ -60,7 +60,7 @@ export const DynamicFormWithTranslations = ({
     i18nContent,
     fieldsToOmit,
     selectedDomain,
-    requiredLangs
+    requiredLangs,
   );
 
   return parsedGroups.map(({ name, langs }, i) => (
@@ -69,9 +69,9 @@ export const DynamicFormWithTranslations = ({
       name={
         name
           ? formatMessage({
-            id: `templates.groups.${name}`,
-            defaultMessage: name,
-          })
+              id: `templates.groups.${name}`,
+              defaultMessage: name,
+            })
           : null
       }
       scope={template?.scope}

package/src/components/FieldGroupDetail.js

@@ -15,7 +15,7 @@ const parseField = (field) => {
         _.prop("fixed_tuple"),
         _.find({ value }),
         _.prop("text"),
-        _.defaultTo(value)
+        _.defaultTo(value),
       )(values);
     return {
       ...field,

package/src/components/FieldGroupWithTranslations.js

@@ -27,7 +27,7 @@ export const FieldGroupWithTranslations = ({
   } = useLanguage();
   const selectedAltLang = altLang || altLangs[0];
   const hasTranslations = hasTranslatableFields(
-    fieldsByLang[defaultLang] || []
+    fieldsByLang[defaultLang] || [],
   );
 
   const getFieldForLang = (field, lang) => {

package/src/components/SelectDynamicFormWithTranslations.js

@@ -1,5 +1,5 @@
 import _ from "lodash/fp";
-import { useEffect, useState } from "react";
+import { useState } from "react";
 import PropTypes from "prop-types";
 import { Grid, Header } from "semantic-ui-react";
 import { TemplateSelector } from "@truedat/core/components";
@@ -29,13 +29,15 @@ export default function SelectDynamicFormWithTranslations({
   showTemplateSectionTitle,
 }) {
   const [thisTemplate, setThisTemplate] = useState();
-  const template = _.isEmpty(selectedTemplate) ? thisTemplate : selectedTemplate;
+  const template = _.isEmpty(selectedTemplate)
+    ? thisTemplate
+    : selectedTemplate;
   const { enabledLangs } = useLanguage();
   const { formatMessage } = useIntl();
   const domainIdsArray = _.isArray(domainIds) ? domainIds : [];
-  const domain = _.size(domainIdsArray) > 0 ? { id: _.head(domainIdsArray) } : null;
+  const domain =
+    _.size(domainIdsArray) > 0 ? { id: _.head(domainIdsArray) } : null;
   const hasDomainSelection = domain !== null;
-  const domainKey = domainIdsArray.join(",");
   const templateTitle =
     template &&
     formatMessage({
@@ -50,17 +52,11 @@ export default function SelectDynamicFormWithTranslations({
     });
   };
 
-  useEffect(() => {
-    if (actionKey === "create") {
-      setThisTemplate(undefined);
-      onChangeTemplate(undefined);
-    }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [domainKey]);
-
   const applyTemplateToLangs = (tpl) => {
     enabledLangs.forEach((lang) => {
-      const content = tpl ? applyTemplate(tpl)(i18nContent[lang] || {}, domain?.id) : {};
+      const content = tpl
+        ? applyTemplate(tpl)(i18nContent[lang] || {}, domain?.id)
+        : {};
       handleChange(content, tpl, lang);
     });
   };
@@ -89,10 +85,13 @@ export default function SelectDynamicFormWithTranslations({
 
   return (
     <>
-      {(beforeSelector || (hasDomainSelection && (!actionKey || actionKey === "create"))) && (
+      {(beforeSelector ||
+        (hasDomainSelection && (!actionKey || actionKey === "create"))) && (
         <Grid stackable>
           <Grid.Row columns={beforeSelector ? 2 : 1} verticalAlign="bottom">
-            {beforeSelector ? <Grid.Column>{beforeSelector}</Grid.Column> : null}
+            {beforeSelector ? (
+              <Grid.Column>{beforeSelector}</Grid.Column>
+            ) : null}
             {hasDomainSelection && (!actionKey || actionKey === "create") ? (
               <Grid.Column>
                 <TemplateSelector

package/src/components/SelectableDynamicForm.js

@@ -16,7 +16,7 @@ export default function SelectableDynamicForm({
   actionKey,
   onChange,
   onNameChange,
-  onTemplateChange = () => { },
+  onTemplateChange = () => {},
   onBlur,
   label,
   placeholder,

package/src/components/__tests__/DynamicFieldValue.spec.js

@@ -57,7 +57,7 @@ describe("<DynamicFieldValue />", () => {
         value={{ value: "foo bar", origin: "user" }}
         type="group"
         widget="dropdown"
-      />
+      />,
     );
 
     expect(rendered.getByText(/foo bar/i)).toBeInTheDocument();
@@ -82,7 +82,7 @@ describe("<DynamicFieldValue />", () => {
         value={{ value: ["foo bar", "bar baz"], origin: "user" }}
         type="group"
         widget="checkbox"
-      />
+      />,
     );
 
     expect(rendered.getByText(/foo bar/i)).toBeInTheDocument();

package/src/components/__tests__/DynamicFormViewer.spec.js

@@ -116,7 +116,7 @@ describe("<DynamicFormViewer />", () => {
     };
 
     const { container } = render(
-      <DynamicFormViewer template={template} content={content} />
+      <DynamicFormViewer template={template} content={content} />,
     );
 
     const reader = container.querySelector(".markdown-reader");
@@ -126,7 +126,7 @@ describe("<DynamicFormViewer />", () => {
 
   it("renders group fields through GroupPreview", () => {
     const rendered = render(
-      <DynamicFormViewer template={groupTemplate} content={groupContent} />
+      <DynamicFormViewer template={groupTemplate} content={groupContent} />,
     );
 
     expect(rendered.getByText(/foo bar/i)).toBeInTheDocument();
@@ -138,7 +138,7 @@ describe("<DynamicFormViewer />", () => {
       <DynamicFormViewer
         template={unnamedGroupTemplate}
         content={groupContent}
-      />
+      />,
     );
 
     expect(rendered.getByText(/foo bar/i)).toBeInTheDocument();
@@ -150,7 +150,7 @@ describe("<DynamicFormViewer />", () => {
     const rendered = render(<DynamicFormViewer content={groupContent} />);
 
     expect(
-      rendered.container.querySelector(".ui.negative.message")
+      rendered.container.querySelector(".ui.negative.message"),
     ).toBeInTheDocument();
   });
 
@@ -159,7 +159,7 @@ describe("<DynamicFormViewer />", () => {
       <DynamicFormViewerFetcher
         template={groupTemplate}
         content={groupContent}
-      />
+      />,
     );
 
     expect(useTemplate).not.toHaveBeenCalled();
@@ -174,7 +174,7 @@ describe("<DynamicFormViewer />", () => {
     });
 
     const rendered = render(
-      <DynamicFormViewerFetcher template="group-template" />
+      <DynamicFormViewerFetcher template="group-template" />,
     );
 
     expect(useTemplate).toHaveBeenCalledWith({
@@ -195,7 +195,7 @@ describe("<DynamicFormViewer />", () => {
         template="group-template"
         domainIds={[1]}
         content={groupContent}
-      />
+      />,
     );
 
     expect(useTemplate).toHaveBeenCalledWith({

package/src/components/__tests__/EditableDynamicFieldValue.spec.js

@@ -31,7 +31,7 @@ describe("<EditableDynamicFieldValue />", () => {
     };
     const wrapper = render(
       <EditableDynamicFieldValue {...props} />,
-      renderOpts
+      renderOpts,
     );
     expect(wrapper).toMatchSnapshot();
   });

package/src/components/__tests__/FieldViewerValue.spec.js

@@ -73,7 +73,7 @@ describe("<FieldViewerValue />", () => {
     }`, () => {
       const { container } = render(<FieldViewerValue {...props} />);
       expect(container).toMatchSnapshot();
-    })
+    }),
   );
 
   it("matches the latest snapshot for multiple", () => {
@@ -94,7 +94,7 @@ describe("<FieldViewerValue />", () => {
         values={{
           processed_groups: ["Group alias"],
         }}
-      />
+      />,
     );
 
     expect(container.textContent).toContain("Group alias");
@@ -109,7 +109,7 @@ describe("<FieldViewerValue />", () => {
         value={markdown}
         label="md"
         widget="markdown"
-      />
+      />,
     );
 
     const reader = container.querySelector(".markdown-reader");
@@ -124,7 +124,7 @@ describe("<FieldViewerValue />", () => {
         value="   "
         label="md"
         widget="markdown"
-      />
+      />,
     );
     expect(container.firstChild).toBeNull();
   });
@@ -144,7 +144,7 @@ describe("<FieldViewerValue />", () => {
             },
           },
         ]}
-      />
+      />,
     );
 
     const tableCell = rendered.container.querySelector("tbody td");
@@ -162,7 +162,7 @@ describe("<FieldViewerValue />", () => {
           table_columns: [{ name: "col1", cardinality: "*", type: "string" }],
         }}
         value={[{}]}
-      />
+      />,
     );
 
     expect(rendered.container.querySelector("tbody td")).toBeEmptyDOMElement();
@@ -170,7 +170,7 @@ describe("<FieldViewerValue />", () => {
 
   it("returns null when the field value is nil", () => {
     const { container } = render(
-      <FieldViewerValue type="string" value={null} />
+      <FieldViewerValue type="string" value={null} />,
     );
 
     expect(container.firstChild).toBeNull();
@@ -185,7 +185,7 @@ describe("<FieldViewerValue />", () => {
         values={{
           processed_groups: [{ name: "Data Owners" }],
         }}
-      />
+      />,
     );
 
     expect(rendered.getByText("Data Owners")).toBeInTheDocument();
@@ -202,7 +202,7 @@ describe("<FieldViewerValue />", () => {
         values={{
           processed_groups: [{ name: "Data Owners" }],
         }}
-      />
+      />,
     );
 
     expect(rendered.getByText("Data Owners")).toBeInTheDocument();
@@ -211,7 +211,7 @@ describe("<FieldViewerValue />", () => {
 
   it("renders user_group values using the raw name when they are not groups", () => {
     const rendered = render(
-      <FieldViewerValue type="user_group" value="user:Jane Doe" multiple />
+      <FieldViewerValue type="user_group" value="user:Jane Doe" multiple />,
     );
 
     expect(rendered.getByText("Jane Doe")).toBeInTheDocument();
@@ -224,11 +224,11 @@ describe("<FieldViewerValue />", () => {
         type="string"
         value="identifier-value"
         widget="identifier"
-      />
+      />,
     );
 
     expect(
-      rendered.container.querySelector(".identifier-value")
+      rendered.container.querySelector(".identifier-value"),
     ).toHaveTextContent("identifier-value");
   });
 });

package/src/components/__tests__/SelectDynamicFormWithTranslations.spec.js

@@ -65,7 +65,7 @@ describe("<SelectDynamicFormWithTranslations />", () => {
         onChangeContent={onChangeContent}
         onChangeTemplate={jest.fn()}
         scope="bg"
-      />
+      />,
     );
 
     await waitFor(() => {

package/src/components/__tests__/SelectableDynamicForm.spec.js

@@ -37,7 +37,7 @@ describe("<SelectableDynamicForm />", () => {
 
       expect(rendered.getByText(/loading/i)).toBeInTheDocument();
       expect(
-        rendered.container.querySelector(".ui.loader")
+        rendered.container.querySelector(".ui.loader"),
       ).not.toBeInTheDocument();
 
       await waitFor(() => {
@@ -48,7 +48,7 @@ describe("<SelectableDynamicForm />", () => {
 
       expect(rendered.queryByText(/loading/i)).not.toBeInTheDocument();
       expect(
-        rendered.container.querySelector(".ui.loader")
+        rendered.container.querySelector(".ui.loader"),
       ).not.toBeInTheDocument();
     });
 
@@ -59,7 +59,7 @@ describe("<SelectableDynamicForm />", () => {
       const user = userEvent.setup({ delay: null });
       await user.click(await rendered.findByText(/template1/i));
       expect(
-        rendered.container.querySelector(".ui.loader")
+        rendered.container.querySelector(".ui.loader"),
       ).not.toBeInTheDocument();
     });
 
@@ -73,7 +73,7 @@ describe("<SelectableDynamicForm />", () => {
       await user.click(templateOption);
 
       expect(
-        rendered.container.querySelector(".ui.loader")
+        rendered.container.querySelector(".ui.loader"),
       ).not.toBeInTheDocument();
     });
 
@@ -86,7 +86,7 @@ describe("<SelectableDynamicForm />", () => {
       await user.click(await rendered.findByText(/template2/i));
 
       expect(
-        rendered.container.querySelector(".ui.loader")
+        rendered.container.querySelector(".ui.loader"),
       ).not.toBeInTheDocument();
     });
 
@@ -100,7 +100,7 @@ describe("<SelectableDynamicForm />", () => {
       };
       const rendered = render(
         <SelectableDynamicForm {...thisProps} />,
-        renderOpts
+        renderOpts,
       );
       await waitForLoad(rendered);
 
@@ -140,7 +140,7 @@ describe("<SelectableDynamicForm />", () => {
       };
       render(<SelectableDynamicForm {...thisProps} />, renderOpts);
       await waitFor(() =>
-        expect(onNameChange).toHaveBeenCalledWith("template1")
+        expect(onNameChange).toHaveBeenCalledWith("template1"),
       );
     });
   });
@@ -169,7 +169,7 @@ describe("<SelectableDynamicForm />", () => {
           onChange={jest.fn()}
           onNameChange={jest.fn()}
         />,
-        renderOpts
+        renderOpts,
       );
       await waitForLoad(rendered);
       expect(rendered.container).toMatchSnapshot();
@@ -200,11 +200,15 @@ describe("<SelectableDynamicForm />", () => {
           {...props}
           disableSelector
           selectedTemplate={selectedTemplate}
-        />
+        />,
       );
       await waitForLoad(rendered);
-      expect(rendered.container.querySelector(".ui.search.selection.dropdown")).not.toBeInTheDocument();
-      expect(rendered.queryByText(/template\.selector\.placeholder/i)).not.toBeInTheDocument();
+      expect(
+        rendered.container.querySelector(".ui.search.selection.dropdown"),
+      ).not.toBeInTheDocument();
+      expect(
+        rendered.queryByText(/template\.selector\.placeholder/i),
+      ).not.toBeInTheDocument();
     });
 
     it("matches snapshot when disableSelector and selectedTemplate are set", async () => {
@@ -213,7 +217,7 @@ describe("<SelectableDynamicForm />", () => {
           {...props}
           disableSelector
           selectedTemplate={selectedTemplate}
-        />
+        />,
       );
       await waitForLoad(rendered);
       expect(rendered.container).toMatchSnapshot();

package/src/components/hierarchies/HierarchyView.js

@@ -15,7 +15,7 @@ const getLastId = _.flow(
   _.pathOr([], "nodes"),
   _.map("id"),
   _.concat(0),
-  _.max
+  _.max,
 );
 
 const validateHierarchy = (hierarchy) => {
@@ -33,7 +33,7 @@ const validateHierarchy = (hierarchy) => {
           ...node,
           error: hasError,
         };
-      })
+      }),
     )(hierarchy),
   };
 
@@ -42,7 +42,7 @@ const validateHierarchy = (hierarchy) => {
     _.flow(
       _.pathOr([], "nodes"),
       _.filter({ error: true }),
-      _.isEmpty
+      _.isEmpty,
     )(validatedHierarchy);
   return [valid, validatedHierarchy];
 };
@@ -75,13 +75,13 @@ export const HierarchyView = ({
   const addedNodes = _.flow(
     _.pathOr([], "nodes"),
     _.filter({ editionStatus: "added" }),
-    _.size
+    _.size,
   )(hierarchy);
 
   const modifiedNodes = _.flow(
     _.pathOr([], "nodes"),
     _.filter({ editionStatus: "dirty" }),
-    _.size
+    _.size,
   )(hierarchy);
 
   const updateMetadata = ({ name, description }) =>
@@ -108,10 +108,10 @@ export const HierarchyView = ({
                   ? "dirty"
                   : node.editionStatus == "dirty"
                     ? null
-                    : node.editionStatus
+                    : node.editionStatus,
               )(originalHierarchy.nodes),
             }
-          : node
+          : node,
       )(hierarchy.nodes),
     });
   };
@@ -132,7 +132,7 @@ export const HierarchyView = ({
     _.flow(
       _.pathOr([], "nodes"),
       _.find({ id: id }),
-      _.isEmpty
+      _.isEmpty,
     )(originalHierarchy)
       ? null
       : setDeletedNodes(deletedNodes + 1);
@@ -166,7 +166,7 @@ export const HierarchyView = ({
                         added: addedNodes,
                         modified: modifiedNodes,
                         deleted: deletedNodes,
-                      }
+                      },
                     )}
                   />
                   <HistoryBackButton

package/src/components/hierarchies/__tests__/HierarchyRoutes.spec.js

@@ -1,6 +1,5 @@
 import React from "react";
 import { render, waitForLoad } from "@truedat/test/render";
-import { HierarchyRoutes } from "../HierarchyRoutes";
 import { useAuthorized } from "@truedat/core/hooks/useAuthorized";
 import {
   HIERARCHIES,
@@ -8,6 +7,7 @@ import {
   HIERARCHY_EDIT,
   HIERARCHY_CREATE,
 } from "@truedat/core/routes";
+import { HierarchyRoutes } from "../HierarchyRoutes";
 
 jest.mock("@truedat/core/hooks/useAuthorized", () => ({
   useAuthorized: jest.fn(() => true),

package/src/components/widgets/DropdownDataLoader.js

@@ -2,7 +2,7 @@ import { lazy } from "react";
 import PropTypes from "prop-types";
 
 const SystemsLoader = lazy(
-  () => import("@truedat/core/components/SystemsLoader")
+  () => import("@truedat/core/components/SystemsLoader"),
 );
 
 export const DropdownDataLoader = ({ type }) =>

package/src/components/widgets/DynamicField.js

@@ -34,14 +34,14 @@ export const DynamicField = ({
     >
       <label>
         {(label && fieldProps.type != "hierarchy") ||
-          (fieldProps.type == "hierarchy" &&
-            (fieldProps.values.hierarchy.min_depth == 0 ||
-              !fieldProps.values.hierarchy.min_depth)) ? (
+        (fieldProps.type == "hierarchy" &&
+          (fieldProps.values.hierarchy.min_depth == 0 ||
+            !fieldProps.values.hierarchy.min_depth)) ? (
           <FormattedMessage id={`fields.${label}`} defaultMessage={label} />
         ) : null}
         {label &&
-          fieldProps.type == "hierarchy" &&
-          fieldProps.values.hierarchy.min_depth > 0 ? (
+        fieldProps.type == "hierarchy" &&
+        fieldProps.values.hierarchy.min_depth > 0 ? (
           <>
             <FormattedMessage id={`fields.${label}`} defaultMessage={label} />
             <Label pointing="left">