@designcrowd/library.brand-page 5.14.8-flacky-nav-21 → 5.14.8

package/src/brand-page-maker/stores/brandpage-maker/brandpage-maker-store.ts

@@ -118,7 +118,6 @@ export interface InitialisePayload {
   businessName: string | undefined;
   logoVariationImageUrls: string[];
   shopConfigData: string | ShopConfigData;
-  isServerRenderOverride?: boolean;
 }
 
 export interface SetIsSavingPayload {
@@ -655,7 +654,6 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
       businessName,
       logoVariationImageUrls,
       shopConfigData,
-      isServerRenderOverride = false,
     }: InitialisePayload) {
       const brandPageContainerStore = useBrandPageContainerStore();
       const userStore = useBrandPageMakerUserStore();
@@ -706,10 +704,9 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
           );
         }
 
-        await migrateBrandPageModel(brandPageModel, {
+        migrateBrandPageModel(brandPageModel, {
           logoImage,
           isUserDraft: this.isUserDraft,
-          isServerRender: isServerRenderOverride,
         });
 
         const userPersonalisation =

package/src/shared/page-settings/store/page-settings-store.ts

@@ -6,7 +6,6 @@ import {
   PageFooterDesktop,
   PageFooterMobile,
   VIEWPORTS,
-  HEADER_DISPLAY,
 } from '../../../brand-page';
 import type {
   Content,

package/src/shared/template-settings/store/template-settings-store.ts

@@ -175,14 +175,13 @@ export const useUiTemplateSettingsStore = defineStore('uiTemplateSettings', {
 
         const templateData = JSON.parse(template.data);
 
-        const { brandPageModel } = await migrateBrandPageModel(
+        const brandPageModel = migrateBrandPageModel(
           new BrandPageModel(templateData),
           {
             logoImage: {
               ...logoSettingsStore.selectedLogoImage,
             },
             isUserDraft: false,
-            isServerRender: false,
           },
         );
 

package/lib/src/brand-page/utils/migration/migration-06.d.ts

@@ -1,4 +0,0 @@
-import type { MigrationData } from '../migration.util';
-export declare const migration06: (migrationResult: MigrationData, options: {
-    isServerRender: boolean;
-}) => Promise<MigrationData>;

package/lib/src/brand-page/utils/migration/migration-06.js

@@ -1,60 +0,0 @@
-import { DEFAULT_LOGO_HEIGHT, DEFAULT_LOGO_WIDTH, PageFooterDesktop, PageFooterMobile, PageHeaderDesktop, PageHeaderMobile, } from '../..';
-import { FabricRenderClient } from '../../../clients/fabric-render-client';
-const calculateConstrainedDimensions = (logoData, maxWidth, maxHeight) => {
-    const maxWidthRem = parseFloat(maxWidth);
-    const maxHeightRem = parseFloat(maxHeight);
-    const logoAspectRatio = logoData.width / logoData.height;
-    let actualHeight;
-    if (logoAspectRatio >= 1) {
-        actualHeight = maxWidthRem / logoAspectRatio;
-    }
-    else {
-        actualHeight = maxHeightRem;
-    }
-    const actualHeightRem = `${Math.ceil(actualHeight)}rem`;
-    return { width: actualHeightRem, height: actualHeightRem };
-};
-export const migration06 = async (migrationResult, options) => {
-    const { brandPageModel } = migrationResult;
-    if (options.isServerRender !== undefined && options.isServerRender) {
-        migrationResult.useOldLogoRenderSize = true;
-        return migrationResult;
-    }
-    try {
-        const fabricClient = FabricRenderClient.getInstance();
-        // Define all sections that need logo data
-        const sections = [
-            { name: 'pageHeaderDesktop', config: brandPageModel.pageHeaderDesktop },
-            { name: 'pageHeaderMobile', config: brandPageModel.pageHeaderMobile },
-            { name: 'pageFooterDesktop', config: brandPageModel.pageFooterDesktop },
-            { name: 'pageFooterMobile', config: brandPageModel.pageFooterMobile },
-        ].filter((section) => section.config);
-        // Fetch logo data for all sections in parallel
-        const logoDataPromises = sections.map(async (section) => {
-            const logoData = section.config?.logoTemplateImage?.logoTemplateToken
-                ? await fabricClient.getLogoSvgUrlByLogoTemplateImageAsync(section.config.logoTemplateImage)
-                : await fabricClient.getLogoSvgUrlByLogoImageAsync(section.config.logoImage);
-            return {
-                sectionName: section.name,
-                logoData,
-                config: section.config,
-            };
-        });
-        const results = await Promise.all(logoDataPromises);
-        // Apply dimensions to each section
-        results.forEach(({ sectionName, logoData, config }) => {
-            if (logoData && logoData.width && logoData.height && config) {
-                const currentMaxWidth = config.logoImageWidth || DEFAULT_LOGO_WIDTH;
-                const currentMaxHeight = config.logoImageHeight || DEFAULT_LOGO_HEIGHT;
-                const dimensions = calculateConstrainedDimensions(logoData, currentMaxWidth, currentMaxHeight);
-                config.logoImageHeight = dimensions.height;
-            }
-        });
-    }
-    catch (error) {
-        console.warn('Migration 06: Failed to fetch logo dimensions:', error);
-        migrationResult.useOldLogoRenderSize = true;
-        return migrationResult;
-    }
-    return migrationResult;
-};

package/src/brand-page/utils/migration/migration-06.test.ts

@@ -1,384 +0,0 @@
-import { BrandPageModel, type BrandPageData } from '../../models';
-import { migration06 } from './migration-06';
-import type { MigrationData } from '../migration.util';
-
-// Helper function to create test data with logo configuration
-const createTestBrandPageData = (
-  overrides: Partial<BrandPageData> = {},
-): BrandPageData => {
-  const defaultLogoConfig = {
-    logoImageWidth: '12rem',
-    logoImageHeight: '9rem',
-    logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-  };
-
-  return {
-    pageHeaderDesktop: defaultLogoConfig,
-    pageHeaderMobile: defaultLogoConfig,
-    pageFooterDesktop: defaultLogoConfig,
-    pageFooterMobile: defaultLogoConfig,
-    ...overrides,
-  } as BrandPageData;
-};
-
-// Helper function to create MigrationData
-const createMigrationData = (brandPageData: BrandPageData): MigrationData => {
-  return {
-    brandPageModel: new BrandPageModel(brandPageData),
-  } as MigrationData;
-};
-
-// Helper function to capture original dimensions
-const captureOriginalDimensions = (migrationData: MigrationData) => {
-  const { brandPageModel } = migrationData;
-
-  return {
-    headerDesktopWidth: brandPageModel.pageHeaderDesktop?.logoImageWidth,
-    headerDesktopHeight: brandPageModel.pageHeaderDesktop?.logoImageHeight,
-    headerMobileWidth: brandPageModel.pageHeaderMobile?.logoImageWidth,
-    headerMobileHeight: brandPageModel.pageHeaderMobile?.logoImageHeight,
-    footerDesktopWidth: brandPageModel.pageFooterDesktop?.logoImageWidth,
-    footerDesktopHeight: brandPageModel.pageFooterDesktop?.logoImageHeight,
-    footerMobileWidth: brandPageModel.pageFooterMobile?.logoImageWidth,
-    footerMobileHeight: brandPageModel.pageFooterMobile?.logoImageHeight,
-  };
-};
-
-// Helper function to verify dimensions unchanged
-const expectDimensionsUnchanged = (
-  result: MigrationData,
-  originalDimensions: ReturnType<typeof captureOriginalDimensions>,
-) => {
-  const { brandPageModel } = result;
-
-  expect(brandPageModel.pageHeaderDesktop?.logoImageWidth).toBe(
-    originalDimensions.headerDesktopWidth,
-  );
-  expect(brandPageModel.pageHeaderDesktop?.logoImageHeight).toBe(
-    originalDimensions.headerDesktopHeight,
-  );
-  expect(brandPageModel.pageHeaderMobile?.logoImageWidth).toBe(
-    originalDimensions.headerMobileWidth,
-  );
-  expect(brandPageModel.pageHeaderMobile?.logoImageHeight).toBe(
-    originalDimensions.headerMobileHeight,
-  );
-  expect(brandPageModel.pageFooterDesktop?.logoImageWidth).toBe(
-    originalDimensions.footerDesktopWidth,
-  );
-  expect(brandPageModel.pageFooterDesktop?.logoImageHeight).toBe(
-    originalDimensions.footerDesktopHeight,
-  );
-  expect(brandPageModel.pageFooterMobile?.logoImageWidth).toBe(
-    originalDimensions.footerMobileWidth,
-  );
-  expect(brandPageModel.pageFooterMobile?.logoImageHeight).toBe(
-    originalDimensions.footerMobileHeight,
-  );
-};
-
-describe('migration06', () => {
-  test('should skip migration when isServerRender is true', async () => {
-    const brandPageData = createTestBrandPageData();
-    const migrationData = createMigrationData(brandPageData);
-    const originalDimensions = captureOriginalDimensions(migrationData);
-
-    const result = await migration06(migrationData, { isServerRender: true });
-
-    // Should set useOldLogoRenderSize flag and return early
-    expect(result.useOldLogoRenderSize).toBe(true);
-    expectDimensionsUnchanged(result, originalDimensions);
-  });
-
-  test('should skip migration when no logo token is available', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: '', logoVariation: 0 }, // Empty token
-      },
-      pageHeaderMobile: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-      pageFooterDesktop: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-      pageFooterMobile: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-    const originalDimensions = captureOriginalDimensions(migrationData);
-
-    const result = await migration06(migrationData, { isServerRender: false });
-
-    // Dimensions should remain unchanged when no logo token
-    expect(result.brandPageModel.pageHeaderDesktop?.logoImageWidth).toBe(
-      originalDimensions.headerDesktopWidth,
-    );
-    expect(result.brandPageModel.pageHeaderDesktop?.logoImageHeight).toBe(
-      originalDimensions.headerDesktopHeight,
-    );
-  });
-
-  test('should use default dimensions when none are provided', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        // No logoImageWidth or logoImageHeight provided
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-      pageHeaderMobile: {
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-      pageFooterDesktop: {
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-      pageFooterMobile: {
-        logoImage: { logoDraftToken: '', logoVariation: 0 },
-      },
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-    const result = await migration06(migrationData, { isServerRender: false });
-
-    // Should not crash and return the migrationData unchanged since no logo token
-    expect(result).toBeDefined();
-    expect(result.brandPageModel.pageHeaderDesktop).toBeDefined();
-    expect(result.brandPageModel.pageHeaderMobile).toBeDefined();
-    expect(result.brandPageModel.pageFooterDesktop).toBeDefined();
-    expect(result.brandPageModel.pageFooterMobile).toBeDefined();
-  });
-
-  test('should handle missing pageHeaderDesktop gracefully', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: undefined,
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-
-    // Should not crash when pageHeaderDesktop is missing
-    await expect(
-      migration06(migrationData, { isServerRender: false }),
-    ).resolves.toBeDefined();
-  });
-
-  test('should return the same migrationData instance when using server render', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '8rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageHeaderMobile: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '8rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageFooterDesktop: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '8rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageFooterMobile: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '8rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-    const result = await migration06(migrationData, { isServerRender: true });
-
-    // Should return the exact same migrationData instance
-    expect(result).toBe(migrationData);
-    expect(result.useOldLogoRenderSize).toBe(true);
-  });
-
-  test('should handle logoTemplateImage when available', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '10rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        logoTemplateImage: { logoTemplateToken: 'template-token' },
-      },
-      pageHeaderMobile: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '10rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        logoTemplateImage: { logoTemplateToken: 'template-token' },
-      },
-      pageFooterDesktop: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '10rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        logoTemplateImage: { logoTemplateToken: 'template-token' },
-      },
-      pageFooterMobile: {
-        logoImageWidth: '10rem',
-        logoImageHeight: '10rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        logoTemplateImage: { logoTemplateToken: 'template-token' },
-      },
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-
-    // The migration should attempt to process the template image
-    // Since we can't mock the API call, we expect it to either succeed or fail gracefully
-    const result = await migration06(migrationData, { isServerRender: false });
-
-    expect(result).toBeDefined();
-    expect(result.brandPageModel.pageHeaderDesktop).toBeDefined();
-  });
-
-  test('should maintain model structure after migration', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        displayStyle: 'desktop-tab-vertical-stack-centered-logo-centered-menu',
-        isLogoVisible: true,
-      },
-      pageHeaderMobile: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        displayStyle: 'mobile-tab-vertical-stack-centered-logo-centered-menu',
-        isLogoVisible: true,
-      },
-      pageFooterDesktop: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        displayStyle: 'centered-menu',
-        isLogoVisible: true,
-      },
-      pageFooterMobile: {
-        logoImageWidth: '12rem',
-        logoImageHeight: '9rem',
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-        displayStyle: 'centered-menu',
-        isLogoVisible: true,
-      },
-      v: 5,
-      brandPageType: 'brandSite',
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-    const result = await migration06(migrationData, { isServerRender: false });
-
-    const { brandPageModel } = result;
-
-    // Should maintain all original properties
-    expect(brandPageModel.pageHeaderDesktop?.displayStyle).toBe(
-      'desktop-tab-vertical-stack-centered-logo-centered-menu',
-    );
-    expect(brandPageModel.pageHeaderDesktop?.isLogoVisible).toBe(true);
-    expect(brandPageModel.pageHeaderMobile?.displayStyle).toBe(
-      'mobile-tab-vertical-stack-centered-logo-centered-menu',
-    );
-    expect(brandPageModel.pageHeaderMobile?.isLogoVisible).toBe(true);
-    expect(brandPageModel.pageFooterDesktop?.displayStyle).toBe(
-      'centered-menu',
-    );
-    expect(brandPageModel.pageFooterDesktop?.isLogoVisible).toBe(true);
-    expect(brandPageModel.pageFooterMobile?.displayStyle).toBe('centered-menu');
-    expect(brandPageModel.pageFooterMobile?.isLogoVisible).toBe(true);
-    expect(brandPageModel.v).toBe(5);
-    expect(brandPageModel.brandPageType).toBe('brandSite');
-
-    // Logo image objects should be preserved
-    expect(brandPageModel.pageHeaderDesktop?.logoImage.logoDraftToken).toBe(
-      'test-token',
-    );
-    expect(brandPageModel.pageHeaderDesktop?.logoImage.logoVariation).toBe(0);
-  });
-
-  test('should calculate dimensions independently for each section with different max constraints', async () => {
-    const brandPageData = createTestBrandPageData({
-      pageHeaderDesktop: {
-        logoImageWidth: '16rem', // 256px max
-        logoImageHeight: '12rem', // 192px max
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageHeaderMobile: {
-        logoImageWidth: '8rem', // 128px max
-        logoImageHeight: '6rem', // 96px max
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageFooterDesktop: {
-        logoImageWidth: '10rem', // 160px max
-        logoImageHeight: '10rem', // 160px max (square)
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-      pageFooterMobile: {
-        logoImageWidth: '6rem', // 96px max
-        logoImageHeight: '8rem', // 128px max (tall)
-        logoImage: { logoDraftToken: 'test-token', logoVariation: 0 },
-      },
-    });
-
-    const migrationData = createMigrationData(brandPageData);
-    const result = await migration06(migrationData, { isServerRender: false });
-
-    const { brandPageModel } = result;
-
-    // Each section should calculate dimensions based on its own constraints
-    // Since we can't mock the API response, we verify that the function doesn't crash
-    // and that each section maintains its structure
-    expect(brandPageModel.pageHeaderDesktop).toBeDefined();
-    expect(brandPageModel.pageHeaderMobile).toBeDefined();
-    expect(brandPageModel.pageFooterDesktop).toBeDefined();
-    expect(brandPageModel.pageFooterMobile).toBeDefined();
-
-    // Verify that different sections can have different dimensions
-    // (This test will pass whether the API succeeds or fails gracefully)
-    expect(typeof brandPageModel.pageHeaderDesktop?.logoImageWidth).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageHeaderDesktop?.logoImageHeight).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageHeaderMobile?.logoImageWidth).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageHeaderMobile?.logoImageHeight).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageFooterDesktop?.logoImageWidth).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageFooterDesktop?.logoImageHeight).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageFooterMobile?.logoImageWidth).toBe(
-      'string',
-    );
-    expect(typeof brandPageModel.pageFooterMobile?.logoImageHeight).toBe(
-      'string',
-    );
-
-    // Logo image objects should be preserved for all sections
-    expect(brandPageModel.pageHeaderDesktop?.logoImage.logoDraftToken).toBe(
-      'test-token',
-    );
-    expect(brandPageModel.pageHeaderMobile?.logoImage.logoDraftToken).toBe(
-      'test-token',
-    );
-    expect(brandPageModel.pageFooterDesktop?.logoImage.logoDraftToken).toBe(
-      'test-token',
-    );
-    expect(brandPageModel.pageFooterMobile?.logoImage.logoDraftToken).toBe(
-      'test-token',
-    );
-  });
-});

package/src/brand-page/utils/migration/migration-06.ts

@@ -1,108 +0,0 @@
-import {
-  DEFAULT_LOGO_HEIGHT,
-  DEFAULT_LOGO_WIDTH,
-  PageFooterDesktop,
-  PageFooterMobile,
-  PageHeaderDesktop,
-  PageHeaderMobile,
-  type BrandPageModel,
-} from '../..';
-import { FabricRenderClient } from '../../../clients/fabric-render-client';
-import type { MigrationData } from '../migration.util';
-
-const calculateConstrainedDimensions = (
-  logoData: { width: number; height: number },
-  maxWidth: string,
-  maxHeight: string,
-) => {
-  const maxWidthRem = parseFloat(maxWidth);
-  const maxHeightRem = parseFloat(maxHeight);
-
-  const logoAspectRatio = logoData.width / logoData.height;
-
-  let actualHeight: number;
-
-  if (logoAspectRatio >= 1) {
-    actualHeight = maxWidthRem / logoAspectRatio;
-  } else {
-    actualHeight = maxHeightRem;
-  }
-
-  const actualHeightRem = `${Math.ceil(actualHeight)}rem`;
-
-  return { width: actualHeightRem, height: actualHeightRem };
-};
-
-export const migration06 = async (
-  migrationResult: MigrationData,
-  options: { isServerRender: boolean },
-): Promise<MigrationData> => {
-  const { brandPageModel } = migrationResult;
-
-  if (options.isServerRender !== undefined && options.isServerRender) {
-    migrationResult.useOldLogoRenderSize = true;
-
-    return migrationResult;
-  }
-
-  try {
-    const fabricClient = FabricRenderClient.getInstance();
-
-    // Define all sections that need logo data
-    const sections: {
-      name: string;
-      config:
-        | PageHeaderDesktop
-        | PageHeaderMobile
-        | PageFooterDesktop
-        | PageFooterMobile;
-    }[] = [
-      { name: 'pageHeaderDesktop', config: brandPageModel.pageHeaderDesktop },
-      { name: 'pageHeaderMobile', config: brandPageModel.pageHeaderMobile },
-      { name: 'pageFooterDesktop', config: brandPageModel.pageFooterDesktop },
-      { name: 'pageFooterMobile', config: brandPageModel.pageFooterMobile },
-    ].filter((section) => section.config);
-
-    // Fetch logo data for all sections in parallel
-    const logoDataPromises = sections.map(async (section) => {
-      const logoData = section.config?.logoTemplateImage?.logoTemplateToken
-        ? await fabricClient.getLogoSvgUrlByLogoTemplateImageAsync(
-            section.config.logoTemplateImage,
-          )
-        : await fabricClient.getLogoSvgUrlByLogoImageAsync(
-            section.config.logoImage,
-          );
-
-      return {
-        sectionName: section.name,
-        logoData,
-        config: section.config,
-      };
-    });
-
-    const results = await Promise.all(logoDataPromises);
-
-    // Apply dimensions to each section
-    results.forEach(({ sectionName, logoData, config }) => {
-      if (logoData && logoData.width && logoData.height && config) {
-        const currentMaxWidth = config.logoImageWidth || DEFAULT_LOGO_WIDTH;
-        const currentMaxHeight = config.logoImageHeight || DEFAULT_LOGO_HEIGHT;
-
-        const dimensions = calculateConstrainedDimensions(
-          logoData,
-          currentMaxWidth,
-          currentMaxHeight,
-        );
-
-        config.logoImageHeight = dimensions.height;
-      }
-    });
-  } catch (error) {
-    console.warn('Migration 06: Failed to fetch logo dimensions:', error);
-    migrationResult.useOldLogoRenderSize = true;
-
-    return migrationResult;
-  }
-
-  return migrationResult;
-};