slice-machine-ui 1.23.2-beta.10 → 1.23.2-beta.11

package/components/ChangesEmptyState/OfflinePage.tsx

@@ -28,7 +28,7 @@ export const OfflinePage = () => {
         <Text
           sx={{ fontSize: "13px", lineHeight: "24px", textAlign: "center" }}
         >
-          You need a stable internet connection to review and push changes
+          Connect to the internet to sync your changes.
         </Text>
       </Flex>
     </Flex>

package/components/ChangesItems/ChangesItems.tsx

@@ -6,19 +6,23 @@ import { ChangesSectionHeader } from "@components/ChangesSectionHeader";
 import { CustomTypeTable } from "@components/CustomTypeTable/changesPage";
 import Grid from "@components/Grid";
 import { ComponentUI } from "@lib/models/common/ComponentUI";
-import { ModelStatusInformation } from "@src/hooks/useModelStatus";
 import ScreenshotChangesModal from "@components/ScreenshotChangesModal";
 import { countMissingScreenshots } from "@src/domain/slice";
 import { useScreenshotChangesModal } from "@src/hooks/useScreenshotChangesModal";
 import { ModelStatus } from "@lib/models/common/ModelStatus";
 import { LocalOrRemoteCustomType } from "@lib/models/common/ModelData";
 import { SharedSliceCard } from "@src/features/slices/sliceCards/SharedSliceCard";
+import { AuthStatus } from "@src/modules/userContext/types";
+import { ModelsStatuses } from "@src/features/sync/getUnSyncChanges";
 
 import { DevCollaborationExperiment } from "./DevCollaborationExperiment";
 
-interface ChangesItemsProps extends ModelStatusInformation {
+interface ChangesItemsProps {
   unSyncedCustomTypes: LocalOrRemoteCustomType[];
   unSyncedSlices: ComponentUI[];
+  modelsStatuses: ModelsStatuses;
+  authStatus: AuthStatus;
+  isOnline: boolean;
 }
 
 export const ChangesItems: React.FC<ChangesItemsProps> = ({

package/components/CustomTypeTable/changesPage.tsx

@@ -1,7 +1,6 @@
 import Link from "next/link";
 import React from "react";
 import { Box, Text } from "theme-ui";
-import { ModelStatusInformation } from "@src/hooks/useModelStatus";
 import { CustomTypeSM } from "@lib/models/common/CustomType";
 import { ModelStatus } from "@lib/models/common/ModelStatus";
 import {
@@ -10,18 +9,26 @@ import {
 } from "@lib/models/common/ModelData";
 import { StatusBadge } from "@src/features/changes/StatusBadge";
 import { CUSTOM_TYPES_CONFIG } from "@src/features/customTypes/customTypesConfig";
+import { AuthStatus } from "@src/modules/userContext/types";
+import { ModelsStatuses } from "@src/features/sync/getUnSyncChanges";
 
-interface CustomTypeTableProps extends ModelStatusInformation {
+interface CustomTypeTableProps {
   customTypes: LocalOrRemoteCustomType[];
+  modelsStatuses: ModelsStatuses;
+  authStatus: AuthStatus;
+  isOnline: boolean;
 }
 
 const firstColumnWidth = "40%";
 const secondColumnWidth = "40%";
 const thirdColumnWidth = "20%";
 
-const CustomTypeChangeRow: React.FC<
-  { ct: CustomTypeSM; status: ModelStatus } & ModelStatusInformation
-> = ({ ct, status, authStatus, isOnline }) => {
+const CustomTypeChangeRow: React.FC<{
+  ct: CustomTypeSM;
+  status: ModelStatus;
+  authStatus: AuthStatus;
+  isOnline: boolean;
+}> = ({ ct, status, authStatus, isOnline }) => {
   return (
     <>
       <Box as={"td"} style={{ width: firstColumnWidth }}>
@@ -80,7 +87,6 @@ export const CustomTypeTable: React.FC<CustomTypeTableProps> = ({
                   status={modelsStatuses.customTypes[customType.local.id]}
                   authStatus={authStatus}
                   isOnline={isOnline}
-                  modelsStatuses={modelsStatuses}
                   key={customType.local.id}
                 />
               </tr>
@@ -97,7 +103,6 @@ export const CustomTypeTable: React.FC<CustomTypeTableProps> = ({
                 status={modelsStatuses.customTypes[customType.remote.id]}
                 authStatus={authStatus}
                 isOnline={isOnline}
-                modelsStatuses={modelsStatuses}
                 key={customType.remote.id}
               />
             </tr>

package/components/DeleteSliceModal/index.tsx

@@ -7,6 +7,7 @@ import useSliceMachineActions from "@src/modules/useSliceMachineActions";
 import Card from "@components/Card";
 import { Button } from "@components/Button";
 import { deleteSlice } from "@src/features/slices/actions/deleteSlice";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
 
 type DeleteSliceModalProps = {
   isOpen: boolean;
@@ -21,6 +22,7 @@ export const DeleteSliceModal: React.FunctionComponent<
 > = ({ sliceId, sliceName, libName, isOpen, onClose }) => {
   const [isDeleting, setIsDeleting] = useState(false);
   const { deleteSliceSuccess } = useSliceMachineActions();
+  const { syncChanges } = useAutoSync();
   const { theme } = useThemeUI();
 
   const onDelete = async () => {
@@ -32,6 +34,7 @@ export const DeleteSliceModal: React.FunctionComponent<
       libraryID: libName,
       onSuccess: () => {
         deleteSliceSuccess(sliceId, libName);
+        syncChanges();
       },
     });
 

package/components/Forms/CreateCustomTypeModal/CreateCustomTypeModal.tsx

@@ -21,6 +21,7 @@ import {
 } from "@src/features/customTypes/actions/createCustomType";
 import { CUSTOM_TYPES_CONFIG } from "@src/features/customTypes/customTypesConfig";
 import { getFormat } from "@src/domain/customType";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
 
 import { InputBox } from "../components/InputBox";
 import { SelectRepeatable } from "../components/SelectRepeatable";
@@ -58,6 +59,7 @@ export const CreateCustomTypeModal: React.FC<CreateCustomTypeModalProps> = ({
   );
   const customTypesMessages = CUSTOM_TYPES_MESSAGES[format];
   const [isIdFieldPristine, setIsIdFieldPristine] = useState(true);
+  const { syncChanges } = useAutoSync();
   const router = useRouter();
 
   const onSubmit = async ({ id, label, repeatable }: FormValues) => {
@@ -86,6 +88,8 @@ export const CreateCustomTypeModal: React.FC<CreateCustomTypeModalProps> = ({
                 }
               : undefined,
         });
+
+        syncChanges();
       },
     });
 

package/components/Forms/CreateSliceModal/CreateSliceModal.tsx

@@ -9,6 +9,7 @@ import ModalFormCard from "@components/ModalFormCard";
 import { createSlice } from "@src/features/slices/actions/createSlice";
 import useSliceMachineActions from "@src/modules/useSliceMachineActions";
 import { getState } from "@src/apiClient";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
 
 import { validateSliceModalValues } from "../formsValidator";
 import { InputBox } from "../components/InputBox";
@@ -30,6 +31,7 @@ export const CreateSliceModal: FC<CreateSliceModalProps> = ({
 }) => {
   const { createSliceSuccess } = useSliceMachineActions();
   const [isCreatingSlice, setIsCreatingSlice] = useState(false);
+  const { syncChanges } = useAutoSync();
 
   const onSubmit = async (values: FormValues) => {
     const sliceName = values.sliceName;
@@ -45,8 +47,8 @@ export const CreateSliceModal: FC<CreateSliceModalProps> = ({
         const serverState = await getState();
         // Update Redux store
         createSliceSuccess(serverState.libraries);
-
         onSuccess(newSlice, libraryName);
+        syncChanges();
       },
     });
   };

package/components/Forms/RenameCustomTypeModal/RenameCustomTypeModal.tsx

@@ -13,6 +13,7 @@ import { CustomTypeFormat } from "@slicemachine/manager";
 import { CUSTOM_TYPES_MESSAGES } from "@src/features/customTypes/customTypesMessages";
 
 import { renameCustomType } from "@src/features/customTypes/actions/renameCustomType";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
 
 interface RenameCustomTypeModalProps {
   isChangesLocal: boolean;
@@ -32,6 +33,7 @@ export const RenameCustomTypeModal: React.FC<RenameCustomTypeModalProps> = ({
   const customTypeName = customType?.label ?? "";
   const customTypeId = customType?.id ?? "";
   const { renameCustomTypeSuccess } = useSliceMachineActions();
+  const { syncChanges } = useAutoSync();
 
   const [isRenaming, setIsRenaming] = useState(false);
 
@@ -46,7 +48,10 @@ export const RenameCustomTypeModal: React.FC<RenameCustomTypeModalProps> = ({
       await renameCustomType({
         model: customType,
         newLabel: values.customTypeName,
-        onSuccess: renameCustomTypeSuccess,
+        onSuccess: (renamedCustomType) => {
+          renameCustomTypeSuccess(renamedCustomType);
+          syncChanges();
+        },
       });
     }
     setIsRenaming(false);

package/components/Forms/RenameSliceModal/RenameSliceModal.tsx

@@ -6,6 +6,7 @@ import { SliceMachineStoreType } from "@src/redux/type";
 import { getLibraries, getRemoteSlices } from "@src/modules/slices";
 import { ComponentUI } from "@lib/models/common/ComponentUI";
 import { renameSlice } from "@src/features/slices/actions/renameSlice";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
 
 import { InputBox } from "../components/InputBox";
 import ModalFormCard from "../../ModalFormCard";
@@ -24,6 +25,7 @@ export const RenameSliceModal: React.FC<RenameSliceModalProps> = ({
   onClose,
 }) => {
   const { renameSliceSuccess } = useSliceMachineActions();
+  const { syncChanges } = useAutoSync();
   const { localLibs, remoteLibs } = useSelector(
     (store: SliceMachineStoreType) => ({
       localLibs: getLibraries(store),
@@ -39,6 +41,7 @@ export const RenameSliceModal: React.FC<RenameSliceModalProps> = ({
         newSliceName: values.sliceName,
         onSuccess: (renamedSlice) => {
           renameSliceSuccess(renamedSlice.from, renamedSlice.model);
+          syncChanges();
         },
       });
 

package/components/LoginModal/index.tsx

@@ -11,7 +11,7 @@ import {
   Text,
 } from "theme-ui";
 import SliceMachineModal from "@components/SliceMachineModal";
-import { checkAuthStatus, startAuth } from "@src/apiClient";
+import { checkAuthStatus, getState, startAuth } from "@src/apiClient";
 import { buildEndpoints } from "@lib/prismic/endpoints";
 import { startPolling } from "@lib/utils/poll";
 import { CheckAuthStatusResponse } from "@models/common/Auth";
@@ -25,6 +25,12 @@ import { getEnvironment } from "@src/modules/environment";
 import useSliceMachineActions from "@src/modules/useSliceMachineActions";
 import preferWroomBase from "@lib/utils/preferWroomBase";
 import { ToasterType } from "@src/modules/toaster";
+import { getUnSyncedChanges } from "@src/features/sync/getUnSyncChanges";
+import { normalizeFrontendCustomTypes } from "@lib/models/common/normalizers/customType";
+import { normalizeFrontendSlices } from "@lib/models/common/normalizers/slices";
+import { AuthStatus } from "@src/modules/userContext/types";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
+import { getActiveEnvironment } from "@src/features/environments/actions/getActiveEnvironment";
 
 interface ValidAuthStatus extends CheckAuthStatusResponse {
   status: "ok";
@@ -42,7 +48,7 @@ const LoginModal: React.FunctionComponent = () => {
       env: getEnvironment(store),
     }),
   );
-
+  const { syncChanges } = useAutoSync();
   const { closeModals, startLoadingLogin, stopLoadingLogin, openToaster } =
     useSliceMachineActions();
 
@@ -72,6 +78,38 @@ const LoginModal: React.FunctionComponent = () => {
       openToaster("Logged in", ToasterType.SUCCESS);
       stopLoadingLogin();
       closeModals();
+
+      const serverState = await getState();
+      const slices = normalizeFrontendSlices(
+        serverState.libraries,
+        serverState.remoteSlices,
+      );
+      const customTypes = Object.values(
+        normalizeFrontendCustomTypes(
+          serverState.customTypes,
+          serverState.remoteCustomTypes,
+        ),
+      );
+      const { changedCustomTypes, changedSlices } = getUnSyncedChanges({
+        authStatus: AuthStatus.AUTHORIZED,
+        customTypes,
+        isOnline: true,
+        libraries: serverState.libraries,
+        slices,
+      });
+      const { activeEnvironment } = await getActiveEnvironment();
+
+      if (
+        activeEnvironment?.kind === "dev" &&
+        (changedCustomTypes.length > 0 || changedSlices.length > 0)
+      ) {
+        syncChanges({
+          environment: activeEnvironment,
+          loggedIn: true,
+          changedCustomTypes,
+          changedSlices,
+        });
+      }
     } catch (e) {
       stopLoadingLogin();
       openToaster("Login failed", ToasterType.ERROR);

package/components/Navigation/ChangesItem.tsx

@@ -0,0 +1,117 @@
+import { type FC } from "react";
+import { useRouter } from "next/router";
+import { Box, Button, Text } from "@prismicio/editor-ui";
+
+import {
+  HoverCard,
+  HoverCardCloseButton,
+  HoverCardDescription,
+  HoverCardMedia,
+  HoverCardTitle,
+} from "@src/components/HoverCard";
+import useSliceMachineActions from "@src/modules/useSliceMachineActions";
+import { useNetwork } from "@src/hooks/useNetwork";
+import { useAuthStatus } from "@src/hooks/useAuthStatus";
+import { useUnSyncChanges } from "@src/features/sync/useUnSyncChanges";
+import { AuthStatus } from "@src/modules/userContext/types";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
+import { AutoSyncStatusIndicator } from "@src/features/sync/components/AutoSyncStatusIndicator";
+
+export const ChangesItem: FC = () => {
+  const { setSeenChangesToolTip } = useSliceMachineActions();
+  const open = useOpenChangesHoverCard();
+  const router = useRouter();
+  const { autoSyncStatus } = useAutoSync();
+
+  const onClose = () => {
+    setSeenChangesToolTip();
+  };
+
+  return (
+    <HoverCard
+      open={open}
+      openDelay={3000}
+      onClose={onClose}
+      side="right"
+      sideOffset={24}
+      collisionPadding={280}
+      trigger={
+        <Box padding={{ bottom: 24 }}>
+          {autoSyncStatus === "failed" ||
+          autoSyncStatus === "synced" ||
+          autoSyncStatus === "syncing" ? (
+            <AutoSyncStatusIndicator autoSyncStatus={autoSyncStatus} />
+          ) : (
+            // TODO(DT-1942): This should be a Button with a link component for
+            // accessibility
+            <Button
+              color="grey"
+              onClick={() => {
+                void router.push("/changes");
+              }}
+              sx={{ width: "100%" }}
+            >
+              <Box alignItems="center" gap={4}>
+                <Text variant="bold">Review changes</Text> <ChangesCount />
+              </Box>
+            </Button>
+          )}
+        </Box>
+      }
+    >
+      <HoverCardTitle>Push your changes</HoverCardTitle>
+      <HoverCardMedia component="image" src="/push.png" />
+      <HoverCardDescription>
+        When you click Save, your changes are saved locally. Then, you can push
+        your models to Prismic from the Changes page.
+      </HoverCardDescription>
+      <HoverCardCloseButton>Got it</HoverCardCloseButton>
+    </HoverCard>
+  );
+};
+
+export const ChangesCount: FC = () => {
+  const isOnline = useNetwork();
+  const authStatus = useAuthStatus();
+  const { unSyncedSlices, unSyncedCustomTypes } = useUnSyncChanges();
+  const numberOfChanges = unSyncedSlices.length + unSyncedCustomTypes.length;
+
+  if (
+    !isOnline ||
+    authStatus === AuthStatus.UNAUTHORIZED ||
+    authStatus === AuthStatus.FORBIDDEN
+  ) {
+    return null;
+  }
+
+  if (numberOfChanges === 0) {
+    return null;
+  }
+
+  const formattedNumberOfChanges = numberOfChanges > 9 ? "+9" : numberOfChanges;
+
+  return (
+    <Box padding={{ inline: 6 }} borderRadius={10} backgroundColor="grey5">
+      <Text color="grey11" variant="small">
+        {formattedNumberOfChanges}
+      </Text>
+    </Box>
+  );
+};
+
+// TODO(DT-1925): Reactivate this feature
+const useOpenChangesHoverCard = () => {
+  // const { hasSeenChangesToolTip, hasSeenSimulatorToolTip } = useSelector(
+  //   (store: SliceMachineStoreType) => ({
+  //     hasSeenChangesToolTip: userHasSeenChangesToolTip(store),
+  //     hasSeenSimulatorToolTip: userHasSeenSimulatorToolTip(store),
+  //   }),
+  // );
+
+  // return (
+  //   !hasSeenChangesToolTip &&
+  //   hasSeenSimulatorToolTip
+  // );
+
+  return false;
+};

package/components/Navigation/Environment.tsx

@@ -1,25 +1,40 @@
+import { useState } from "react";
 import {
   Environment as EnvironmentType,
   isUnauthenticatedError,
   isUnauthorizedError,
 } from "@slicemachine/manager/client";
 
-import { telemetry } from "@src/apiClient";
+import { getState, telemetry } from "@src/apiClient";
 import { useEnvironments } from "@src/features/environments/useEnvironments";
 import { setEnvironment } from "@src/features/environments/actions/setEnvironment";
 import { useActiveEnvironment } from "@src/features/environments/useActiveEnvironment";
-import { getLegacySliceMachineState } from "@src/features/legacyState/actions/getLegacySliceMachineState";
 import useSliceMachineActions from "@src/modules/useSliceMachineActions";
 import { SideNavEnvironmentSelector } from "@src/components/SideNav";
 import { useNetwork } from "@src/hooks/useNetwork";
+import { useAutoSync } from "@src/features/sync/AutoSyncProvider";
+import { normalizeFrontendSlices } from "@lib/models/common/normalizers/slices";
+import { normalizeFrontendCustomTypes } from "@lib/models/common/normalizers/customType";
+import { getUnSyncedChanges } from "@src/features/sync/getUnSyncChanges";
+import { useAuthStatus } from "@src/hooks/useAuthStatus";
 
 export function Environment() {
   const { environments, error: useEnvironmentsError } = useEnvironments();
   const { activeEnvironment } = useActiveEnvironment();
   const { refreshState, openLoginModal } = useSliceMachineActions();
+  const { syncChanges } = useAutoSync();
   const isOnline = useNetwork();
+  const authStatus = useAuthStatus();
+  const [isSwitchingEnv, setIsSwitchingEnv] = useState(false);
+  const { autoSyncStatus } = useAutoSync();
 
   async function onSelect(environment: EnvironmentType) {
+    if (activeEnvironment?.name === environment.name) {
+      return;
+    }
+
+    setIsSwitchingEnv(true);
+
     void telemetry.track({
       event: "environment:switch",
       domain: environment.domain,
@@ -27,9 +42,39 @@ export function Environment() {
 
     await setEnvironment(environment);
 
-    const legacySliceMachineState = await getLegacySliceMachineState();
+    const serverState = await getState();
+    refreshState(serverState);
+
+    const slices = normalizeFrontendSlices(
+      serverState.libraries,
+      serverState.remoteSlices,
+    );
+    const customTypes = Object.values(
+      normalizeFrontendCustomTypes(
+        serverState.customTypes,
+        serverState.remoteCustomTypes,
+      ),
+    );
+    const { changedCustomTypes, changedSlices } = getUnSyncedChanges({
+      authStatus,
+      customTypes,
+      isOnline,
+      libraries: serverState.libraries,
+      slices,
+    });
+
+    if (
+      environment.kind === "dev" &&
+      (changedCustomTypes.length > 0 || changedSlices.length > 0)
+    ) {
+      syncChanges({
+        environment,
+        changedCustomTypes,
+        changedSlices,
+      });
+    }
 
-    refreshState(legacySliceMachineState);
+    setIsSwitchingEnv(false);
   }
 
   if (!isOnline) {
@@ -42,6 +87,8 @@ export function Environment() {
         environments={environments}
         activeEnvironment={activeEnvironment}
         onSelect={onSelect}
+        disabled={isSwitchingEnv || autoSyncStatus === "syncing"}
+        loading={isSwitchingEnv}
       />
     );
   }

package/components/Navigation/index.css.ts

@@ -1,6 +1,5 @@
 import { sprinkles } from "@prismicio/editor-ui";
 
 export const environmentDivider = sprinkles({
-  marginTop: 16,
-  marginBottom: 32,
+  marginBlock: 16,
 });

package/components/Navigation/index.test.tsx

@@ -131,10 +131,8 @@ describe("Side Navigation", () => {
   test.skip.each([
     ["Page types", "/"],
     ["Custom types", "/custom-types"],
-    ["Changes", "/changes"],
-
-    ["Changelog", "/changelog"],
     ["Slices", "/slices"],
+    ["Changelog", "/changelog"],
     // TODO: once we have a plan fo how to display individual libraries change this
     // ["Slices/a", "/slices#slices/a"],
     // ["Slices/b", "/slices#slices/b"]
@@ -156,95 +154,17 @@ describe("Side Navigation", () => {
     },
   );
 
-  test("should display the number of changes on the 'Changes' item", async () => {
+  test("should display the number of changes on the 'Review changes' item", async () => {
     renderSideNavigation({ canUpdate: true, hasChanges: true });
 
-    expect(await screen.findByText("Changes")).toBeVisible();
+    expect(await screen.findByText("Review changes")).toBeVisible();
     const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
+      .getByText("Review changes")
+      .closest("button") as HTMLButtonElement;
 
     expect(within(changesItem).getByText("1")).toBeVisible();
   });
 
-  test("should not display the number of changes or 'Disconnect' on the 'Changes' item", async () => {
-    renderSideNavigation({ canUpdate: true, hasChanges: false });
-
-    expect(await screen.findByText("Changes")).toBeVisible();
-    const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
-
-    expect(within(changesItem).queryByText("1")).not.toBeInTheDocument();
-    expect(
-      within(changesItem).queryByText("Logged out"),
-    ).not.toBeInTheDocument();
-  });
-
-  test("should display the information that user is disconnected on the 'Changes' item when unauthorized", async () => {
-    renderSideNavigation({
-      canUpdate: true,
-      hasChanges: true,
-      authStatus: AuthStatus.UNAUTHORIZED,
-    });
-
-    expect(await screen.findByText("Changes")).toBeVisible();
-    const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
-
-    expect(within(changesItem).getByText("Logged out")).toBeVisible();
-  });
-
-  test("should display the information that user is disconnected on the 'Changes' item when forbidden", async () => {
-    renderSideNavigation({
-      canUpdate: true,
-      hasChanges: true,
-      authStatus: AuthStatus.FORBIDDEN,
-    });
-
-    expect(await screen.findByText("Changes")).toBeVisible();
-    const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
-
-    expect(within(changesItem).getByText("Logged out")).toBeVisible();
-  });
-
-  test("should display the information that user is disconnected on the 'Changes' item when offline", async () => {
-    vi.spyOn(navigator, "onLine", "get").mockReturnValueOnce(false);
-
-    renderSideNavigation({
-      canUpdate: true,
-      hasChanges: true,
-    });
-
-    expect(await screen.findByText("Changes")).toBeVisible();
-    const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
-
-    expect(within(changesItem).getByText("Logged out")).toBeVisible();
-  });
-
-  test("should not display the information that user is disconnected when user is online", async () => {
-    vi.spyOn(navigator, "onLine", "get").mockReturnValueOnce(true);
-
-    renderSideNavigation({
-      canUpdate: true,
-      hasChanges: true,
-    });
-
-    expect(await screen.findByText("Changes")).toBeVisible();
-    const changesItem = screen
-      .getByText("Changes")
-      .closest("a") as HTMLAnchorElement;
-
-    expect(
-      within(changesItem).queryByText("Logged out"),
-    ).not.toBeInTheDocument();
-  });
-
   test("Video Item with next", async (ctx) => {
     const adapter = createTestPlugin({
       meta: {

package/components/Navigation/index.tsx

@@ -11,7 +11,6 @@ import { MathPlusIcon } from "@src/icons/MathPlusIcon";
 import { SettingsIcon } from "@src/icons/SettingsIcon";
 import { CUSTOM_TYPES_CONFIG } from "@src/features/customTypes/customTypesConfig";
 import {
-  SideNavSeparator,
   SideNavLink,
   SideNavListItem,
   SideNavList,
@@ -30,7 +29,7 @@ import { CUSTOM_TYPES_MESSAGES } from "@src/features/customTypes/customTypesMess
 import { useGitIntegrationExperiment } from "@src/features/settings/git/useGitIntegrationExperiment";
 import { useRepositoryInformation } from "@src/hooks/useRepositoryInformation";
 
-import { ChangesListItem } from "./ChangesListItem";
+import { ChangesItem } from "./ChangesItem";
 import { Environment } from "./Environment";
 
 import * as styles from "./index.css";
@@ -76,6 +75,8 @@ const Navigation: FC = () => {
         href={repositoryUrl}
       />
 
+      <ChangesItem />
+
       <SideNavList>
         <SideNavListItem>
           <SideNavLink
@@ -107,12 +108,6 @@ const Navigation: FC = () => {
           />
         </SideNavListItem>
 
-        <SideNavSeparator />
-
-        <ChangesListItem />
-
-        <SideNavSeparator />
-
         <SideNavListItem>
           <SideNavLink
             title="Slices"