@truedat/core 8.11.0 → 8.11.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/core",
-  "version": "8.11.0",
+  "version": "8.11.1",
   "description": "Truedat Web Core",
   "sideEffects": [
     "**/*.css",
@@ -98,5 +98,5 @@
     "swr": "^2.3.3",
     "turndown": "^7.2.2"
   },
-  "gitHead": "89b12f5fa7ff2d0fb790d88830938ccff9cb3d91"
+  "gitHead": "c8c61ac1ab65f2fcda5412fe37591f784a71a5c1"
 }

package/src/components/GlossaryMenu.js

@@ -8,7 +8,6 @@ import { linkTo } from "@truedat/core/routes";
 import { makeGetSubscopes } from "@truedat/core/selectors/subscopedTemplates";
 import { getConceptSubscope } from "@truedat/core/selectors/getConceptSubscope";
 import { getSidemenuGlossarySubscopes } from "@truedat/core/selectors/getSidemenuGlossarySubscopes";
-import TemplatesLoader from "@truedat/core/components/TemplatesLoader";
 import { useAuthorized } from "../hooks";
 import {
   CONCEPTS,
@@ -270,7 +269,6 @@ export const GlossaryMenu = ({ bgSubscopes, rootSubscopes }) => {
 
   return (
     <>
-      <TemplatesLoader />
       <Submenu
         icon={defaultIcon}
         name="glossary"

package/src/components/SideMenu.js

@@ -14,6 +14,7 @@ import SidebarToggle from "./SidebarToggle";
 import TaxonomyMenu from "./TaxonomyMenu";
 import GrantMenu from "./GrantMenu";
 import MembersMenu from "./MembersMenu";
+import TemplatesLoader from "./TemplatesLoader";
 
 export const SideMenu = ({
   animation = "push",
@@ -35,6 +36,7 @@ export const SideMenu = ({
         animation={animation}
       >
         <SearchMenu />
+        <TemplatesLoader />
         <GlossaryMenu />
         <CatalogMenu />
         <GrantMenu />

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

@@ -1,5 +1,6 @@
 import { render } from "@truedat/test/render";
 import GlossaryMenu from "../GlossaryMenu";
+import { fetchTemplates } from "@truedat/core/routines";
 
 jest.mock("../../hooks", () => ({
   useActiveRoutes: jest.fn(() => true),
@@ -77,4 +78,20 @@ describe("<GlossaryMenu />", () => {
     expect(queryByRole("option", { name: "subscope1" })).toBeInTheDocument();
     expect(queryByRole("option", { name: "subscope2" })).toBeInTheDocument();
   });
+
+  it("does not dispatch fetchTemplates on mount (templates preload moved to SideMenu)", () => {
+    const dispatched = [];
+    render(<GlossaryMenu />, {
+      ...renderOpts,
+      dispatch: jest.fn((action) => {
+        dispatched.push(action);
+        return action;
+      }),
+    });
+
+    const fetchTemplatesCalls = dispatched.filter(
+      (action) => action && action.type === fetchTemplates.TRIGGER,
+    );
+    expect(fetchTemplatesCalls).toHaveLength(0);
+  });
 });

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

@@ -1,6 +1,7 @@
 import { render, waitForLoad } from "@truedat/test/render";
 import { CATALOG_VIEW_CONFIGS_QUERY } from "@truedat/core/api/queries";
 import SideMenu from "../SideMenu";
+import { fetchTemplates } from "@truedat/core/routines";
 
 jest.mock("@truedat/core/hooks", () => ({
   useActiveRoutes: jest.fn(() => true),
@@ -46,4 +47,63 @@ describe("<SideMenu />", () => {
     await waitForLoad(rendered);
     expect(rendered.container).toMatchSnapshot();
   });
+
+  describe("global templates preload", () => {
+    let dispatched;
+    let useAuthorizedMock;
+
+    beforeEach(() => {
+      dispatched = [];
+      useAuthorizedMock = require("@truedat/core/hooks").useAuthorized;
+      useAuthorizedMock.mockReturnValue(true);
+    });
+
+    const renderWithDispatch = () =>
+      render(
+        <SideMenu>
+          <p>Hello</p>
+        </SideMenu>,
+        {
+          ...renderOpts,
+          dispatch: jest.fn((action) => {
+            dispatched.push(action);
+            return action;
+          }),
+        },
+      );
+
+    it("dispatches fetchTemplates with no scope on mount", async () => {
+      const rendered = renderWithDispatch();
+      await waitForLoad(rendered);
+      expect(dispatched).toContainEqual(
+        expect.objectContaining({
+          type: fetchTemplates.TRIGGER,
+          payload: { scope: undefined },
+        }),
+      );
+    });
+
+    it("dispatches fetchTemplates even when user lacks business_glossary_view", async () => {
+      useAuthorizedMock.mockImplementation((allowedRoles) => {
+        const roles = Array.isArray(allowedRoles)
+          ? allowedRoles
+          : [allowedRoles];
+        // Deny business_glossary_view/management but allow everything else.
+        const blocked = new Set([
+          "business_glossary_view",
+          "business_glossary_management",
+        ]);
+        return roles.some((r) => r && !blocked.has(r));
+      });
+
+      const rendered = renderWithDispatch();
+      await waitForLoad(rendered);
+      expect(dispatched).toContainEqual(
+        expect.objectContaining({
+          type: fetchTemplates.TRIGGER,
+          payload: { scope: undefined },
+        }),
+      );
+    });
+  });
 });