fix peek error circular reference (#18307)

* move error viewer modal

* late import for peek error

Author: nielslyngsoe

src/Umbraco.Web.UI.Client/src/packages/core/manifests.ts

@@ -11,7 +11,6 @@ import { manifests as iconRegistryManifests } from './icon-registry/manifests.js
 import { manifests as localizationManifests } from './localization/manifests.js';
 import { manifests as menuManifests } from './menu/manifests.js';
 import { manifests as modalManifests } from './modal/manifests.js';
-import { manifests as notificationManifests } from './notification/manifests.js';
 import { manifests as pickerManifests } from './picker/manifests.js';
 import { manifests as propertyActionManifests } from './property-action/manifests.js';
 import { manifests as propertyEditorManifests } from './property-editor/manifests.js';
@@ -41,7 +40,6 @@ export const manifests: Array<UmbExtensionManifest | UmbExtensionManifestKind> =
 	...localizationManifests,
 	...menuManifests,
 	...modalManifests,
-	...notificationManifests,
 	...pickerManifests,
 	...propertyActionManifests,
 	...propertyEditorManifests,

src/Umbraco.Web.UI.Client/src/packages/core/notification/modals/error-viewer/error-viewer-modal.element.ts renamed to src/Umbraco.Web.UI.Client/src/packages/core/modal/common/error-viewer/error-viewer-modal.element.ts

@@ -1,5 +1,5 @@
+import type { UmbPeekErrorArgs } from '../../../notification/types.js';
 import { UmbModalToken } from '@umbraco-cms/backoffice/modal';
-import type { UmbPeekErrorArgs } from '../../types.js';
 
 // eslint-disable-next-line @typescript-eslint/no-empty-object-type
 export interface UmbErrorViewerModalData extends UmbPeekErrorArgs {}

src/Umbraco.Web.UI.Client/src/packages/core/notification/modals/error-viewer/error-viewer-modal.token.ts renamed to src/Umbraco.Web.UI.Client/src/packages/core/modal/common/error-viewer/error-viewer-modal.token.ts

@@ -0,0 +1,10 @@
+import type { ManifestModal } from '@umbraco-cms/backoffice/modal';
+
+export const manifests: Array<ManifestModal> = [
+	{
+		type: 'modal',
+		alias: 'Umb.Modal.ErrorViewer',
+		name: 'Error Viewer Modal',
+		element: () => import('./error-viewer-modal.element.js'),
+	},
+];

src/Umbraco.Web.UI.Client/src/packages/core/notification/modals/error-viewer/index.ts renamed to src/Umbraco.Web.UI.Client/src/packages/core/modal/common/error-viewer/index.ts

@@ -1,3 +1,4 @@
 export * from './confirm/index.js';
 export * from './discard-changes/index.js';
+export * from './error-viewer/index.js';
 export * from './item-picker/index.js';

src/Umbraco.Web.UI.Client/src/packages/core/modal/common/error-viewer/manifest.ts

@@ -1,9 +1,11 @@
 import { manifests as confirmManifests } from './confirm/manifests.js';
 import { manifests as discardChangesManifests } from './discard-changes/manifests.js';
+import { manifests as errorViewerManifests } from './error-viewer/manifest.js';
 import { manifests as itemPickerManifests } from './item-picker/manifests.js';
 
 export const manifests: Array<UmbExtensionManifest> = [
 	...confirmManifests,
 	...discardChangesManifests,
+	...errorViewerManifests,
 	...itemPickerManifests,
 ];

src/Umbraco.Web.UI.Client/src/packages/core/modal/common/index.ts

@@ -1,9 +1,8 @@
-import { customElement, html, ifDefined, nothing, property } from '@umbraco-cms/backoffice/external/lit';
-import { UmbLitElement } from '@umbraco-cms/backoffice/lit-element';
 import type { UmbNotificationHandler } from '../../notification-handler.js';
-import { UMB_MODAL_MANAGER_CONTEXT } from '@umbraco-cms/backoffice/modal';
 import type { UmbPeekErrorArgs } from '../../types.js';
-import { UMB_ERROR_VIEWER_MODAL } from '../../index.js';
+import { customElement, html, ifDefined, nothing, property } from '@umbraco-cms/backoffice/external/lit';
+import { UmbLitElement } from '@umbraco-cms/backoffice/lit-element';
+import { UMB_ERROR_VIEWER_MODAL, UMB_MODAL_MANAGER_CONTEXT } from '@umbraco-cms/backoffice/modal';
 
 @customElement('umb-peek-error-notification')
 export class UmbPeekErrorNotificationElement extends UmbLitElement {

src/Umbraco.Web.UI.Client/src/packages/core/modal/common/manifests.ts

@@ -2,7 +2,6 @@ import './layouts/default/index.js';
 export * from './controllers/peek-error/index.js';
 export * from './extractUmbNotificationColor.function.js';
 export * from './isUmbNotifications.function.js';
-export * from './modals/error-viewer/index.js';
 export * from './notification-handler.js';
 export * from './notification.context.js';