@open-mercato/core 0.4.2-canary-15c0b23a3a → 0.4.2-canary-da2b080494

package/src/modules/dashboards/services/widgetDataService.ts

@@ -1,10 +1,6 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
 import type { CacheStrategy } from '@open-mercato/cache'
 import { createHash } from 'node:crypto'
-import { decryptWithAesGcm } from '@open-mercato/shared/lib/encryption/aes'
-import { resolveTenantEncryptionService } from '@open-mercato/shared/lib/encryption/customFieldValues'
-import { resolveEntityIdFromMetadata } from '@open-mercato/shared/lib/encryption/entityIds'
-import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import {
   type DateRangePreset,
   resolveDateRange,
@@ -286,75 +282,6 @@ export class WidgetDataService {
     assertSafeIdentifier(config.idColumn, 'id column')
     assertSafeIdentifier(config.labelColumn, 'label column')
 
-    const meta = this.resolveEntityMetadata(config.table)
-    const idProp = meta ? this.resolveEntityPropertyName(meta, config.idColumn) : null
-    const labelProp = meta ? this.resolveEntityPropertyName(meta, config.labelColumn) : null
-    const tenantProp = meta
-      ? (this.resolveEntityPropertyName(meta, 'tenant_id') ?? this.resolveEntityPropertyName(meta, 'tenantId'))
-      : null
-    const organizationProp = meta
-      ? (this.resolveEntityPropertyName(meta, 'organization_id') ?? this.resolveEntityPropertyName(meta, 'organizationId'))
-      : null
-    const entityName = meta ? ((meta as any).class ?? meta.className ?? meta.name) : null
-
-    if (meta && idProp && labelProp && tenantProp && entityName) {
-      const where: Record<string, unknown> = {
-        [idProp]: { $in: uniqueIds },
-        [tenantProp]: this.scope.tenantId,
-      }
-      if (organizationProp && this.scope.organizationIds && this.scope.organizationIds.length > 0) {
-        where[organizationProp] = { $in: this.scope.organizationIds }
-      }
-
-      try {
-        const records = await findWithDecryption(
-          this.em,
-          entityName,
-          where,
-          { fields: [idProp, labelProp, tenantProp, organizationProp].filter(Boolean) },
-          { tenantId: this.scope.tenantId, organizationId: this.resolveOrganizationId() },
-        )
-
-        const encryptionService = resolveTenantEncryptionService(this.em as any)
-        const dek = encryptionService?.isEnabled() ? await encryptionService.getDek(this.scope.tenantId) : null
-        let hasEncryptedLabels = false
-        let hasDecryptedLabels = false
-
-        const labelMap = new Map<string, string>()
-        for (const record of records as Array<Record<string, unknown>>) {
-          const id = record[idProp]
-          let labelValue = record[labelProp]
-          if (typeof labelValue === 'string' && this.isEncryptedPayload(labelValue)) {
-            hasEncryptedLabels = true
-            if (dek?.key) {
-              const decrypted = this.decryptWithDek(labelValue, dek.key)
-              if (decrypted !== null) {
-                labelValue = decrypted
-                hasDecryptedLabels = true
-              }
-            }
-          } else if (labelValue != null && labelValue !== '') {
-            hasDecryptedLabels = true
-          }
-
-          if (typeof id === 'string' && labelValue != null && labelValue !== '') {
-            labelMap.set(id, String(labelValue))
-          }
-        }
-
-        if (labelMap.size > 0 && (!hasEncryptedLabels || hasDecryptedLabels)) {
-          return data.map((item) => ({
-            ...item,
-            groupLabel: typeof item.groupKey === 'string' && labelMap.has(item.groupKey)
-              ? labelMap.get(item.groupKey)!
-              : undefined,
-          }))
-        }
-      } catch {
-        // fall through to SQL resolution
-      }
-    }
-
     const clauses = [`"${config.idColumn}" = ANY(?::uuid[])`, 'tenant_id = ?']
     const params: unknown[] = [`{${uniqueIds.join(',')}}`, this.scope.tenantId]
 
@@ -363,43 +290,17 @@ export class WidgetDataService {
       params.push(`{${this.scope.organizationIds.join(',')}}`)
     }
 
-    const sql = `SELECT "${config.idColumn}" as id, "${config.labelColumn}" as label, tenant_id, organization_id FROM "${config.table}" WHERE ${clauses.join(
+    const sql = `SELECT "${config.idColumn}" as id, "${config.labelColumn}" as label FROM "${config.table}" WHERE ${clauses.join(
       ' AND ',
     )}`
 
     try {
       const labelRows = await this.em.getConnection().execute(sql, params)
-      const entityId = this.resolveEntityId(meta)
-      const encryptionService = resolveTenantEncryptionService(this.em as any)
-      const organizationId = this.resolveOrganizationId()
-      const dek = encryptionService?.isEnabled() ? await encryptionService.getDek(this.scope.tenantId) : null
 
       const labelMap = new Map<string, string>()
-      for (const row of labelRows as Array<{ id: string; label: string | null; tenant_id?: string | null; organization_id?: string | null }>) {
-        let labelValue = row.label
-        if (entityId && encryptionService?.isEnabled() && labelValue != null) {
-          const rowOrgId = row.organization_id ?? organizationId ?? null
-          const decrypted = await encryptionService.decryptEntityPayload(
-            entityId,
-            { [config.labelColumn]: labelValue },
-            this.scope.tenantId,
-            rowOrgId,
-          )
-          const resolved = decrypted[config.labelColumn]
-          if (typeof resolved === 'string' || typeof resolved === 'number') {
-            labelValue = String(resolved)
-          }
-        }
-
-        if (labelValue && dek?.key && this.isEncryptedPayload(labelValue)) {
-          const decrypted = this.decryptWithDek(labelValue, dek.key)
-          if (decrypted !== null) {
-            labelValue = decrypted
-          }
-        }
-
-        if (row.id && labelValue != null && labelValue !== '') {
-          labelMap.set(row.id, labelValue)
+      for (const row of labelRows as Array<{ id: string; label: string | null }>) {
+        if (row.id && row.label != null && row.label !== '') {
+          labelMap.set(row.id, row.label)
         }
       }
 
@@ -416,60 +317,6 @@ export class WidgetDataService {
       }))
     }
   }
-
-  private resolveOrganizationId(): string | null {
-    if (!this.scope.organizationIds || this.scope.organizationIds.length !== 1) return null
-    return this.scope.organizationIds[0] ?? null
-  }
-
-  private resolveEntityMetadata(tableName: string): Record<string, any> | null {
-    const registry = (this.em as any)?.getMetadata?.()
-    if (!registry) return null
-    const entries =
-      (typeof registry.getAll === 'function' && registry.getAll()) ||
-      (Array.isArray(registry.metadata) ? registry.metadata : Object.values(registry.metadata ?? {}))
-    const metas = Array.isArray(entries) ? entries : Object.values(entries ?? {})
-    const match = metas.find((meta: any) => {
-      const table = meta?.tableName ?? meta?.collection
-      if (typeof table !== 'string') return false
-      if (table === tableName) return true
-      return table.split('.').pop() === tableName
-    })
-    return match ?? null
-  }
-
-  private resolveEntityPropertyName(meta: Record<string, any>, columnName: string): string | null {
-    const properties = meta?.properties ? Object.values(meta.properties) : []
-    for (const prop of properties as Array<Record<string, any>>) {
-      const fieldName = prop?.fieldName
-      const fieldNames = prop?.fieldNames
-      if (typeof fieldName === 'string' && fieldName === columnName) return prop?.name ?? null
-      if (Array.isArray(fieldNames) && fieldNames.includes(columnName)) return prop?.name ?? null
-      if (prop?.name === columnName) return prop?.name ?? null
-    }
-    return null
-  }
-
-  private resolveEntityId(meta: Record<string, any> | null): string | null {
-    if (!meta) return null
-    try {
-      return resolveEntityIdFromMetadata(meta as any)
-    } catch {
-      return null
-    }
-  }
-
-  private isEncryptedPayload(value: string): boolean {
-    const parts = value.split(':')
-    return parts.length === 4 && parts[3] === 'v1'
-  }
-
-  private decryptWithDek(value: string, dek: string): string | null {
-    const first = decryptWithAesGcm(value, dek)
-    if (first === null) return null
-    if (!this.isEncryptedPayload(first)) return first
-    return decryptWithAesGcm(first, dek) ?? first
-  }
 }
 
 export function createWidgetDataService(

package/src/modules/dictionaries/components/DictionaryTable.tsx

@@ -171,14 +171,12 @@ export function DictionaryTable({
               const items: RowActionItem[] = []
               if (onEdit) {
                 items.push({
-                  id: 'edit',
                   label: translations.editLabel,
                   onSelect: () => onEdit(entry),
                 })
               }
               if (onDelete) {
                 items.push({
-                  id: 'delete',
                   label: translations.deleteLabel,
                   onSelect: () => onDelete(entry),
                   destructive: true,

package/src/modules/directory/backend/directory/organizations/page.tsx

@@ -235,8 +235,8 @@ export default function DirectoryOrganizationsPage() {
             canManage ? (
               <RowActions
                 items={[
-                  { id: 'edit', label: t('directory.organizations.list.actions.edit', 'Edit'), href: `/backend/directory/organizations/${row.id}/edit` },
-                  { id: 'delete', label: t('directory.organizations.list.actions.delete', 'Delete'), destructive: true, onSelect: () => handleDelete(row) },
+                  { label: t('directory.organizations.list.actions.edit', 'Edit'), href: `/backend/directory/organizations/${row.id}/edit` },
+                  { label: t('directory.organizations.list.actions.delete', 'Delete'), destructive: true, onSelect: () => handleDelete(row) },
                 ]}
               />
             ) : null

package/src/modules/directory/backend/directory/tenants/page.tsx

@@ -160,8 +160,8 @@ export default function DirectoryTenantsPage() {
             canManage ? (
               <RowActions
                 items={[
-                  { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/directory/tenants/${row.id}/edit` },
-                  { id: 'delete', label: t('common.delete', 'Delete'), destructive: true, onSelect: () => handleDelete(row) },
+                  { label: t('common.edit', 'Edit'), href: `/backend/directory/tenants/${row.id}/edit` },
+                  { label: t('common.delete', 'Delete'), destructive: true, onSelect: () => handleDelete(row) },
                 ]}
               />
             ) : null

package/src/modules/entities/backend/entities/user/[entityId]/records/page.tsx

@@ -338,8 +338,8 @@ export RECORD_ID="<record uuid>"`}</code></pre>
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: 'Edit', href: `/backend/entities/user/${encodeURIComponent(entityId)}/records/${encodeURIComponent(String((row as any).id))}` },
-                { id: 'delete', label: 'Delete', destructive: true, onSelect: async () => {
+                { label: 'Edit', href: `/backend/entities/user/${encodeURIComponent(entityId)}/records/${encodeURIComponent(String((row as any).id))}` },
+                { label: 'Delete', destructive: true, onSelect: async () => {
                   try {
                     if (typeof window !== 'undefined') {
                       const ok = window.confirm('Delete this record?')

package/src/modules/entities/components/SystemEntitiesTable.tsx

@@ -88,7 +88,7 @@ export default function SystemEntitiesTable() {
       rowActions={(row) => (
         <RowActions
           items={[
-            { id: 'edit', label: 'Edit', href: `/backend/entities/system/${encodeURIComponent(row.entityId)}` },
+            { label: 'Edit', href: `/backend/entities/system/${encodeURIComponent(row.entityId)}` },
           ]}
         />
       )}

package/src/modules/entities/components/UserEntitiesTable.tsx

@@ -110,8 +110,8 @@ export default function UserEntitiesTable() {
       rowActions={(row) => (
         <RowActions
           items={[
-            { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/entities/user/${encodeURIComponent(row.entityId)}` },
-            { id: 'show-records', label: t('entities.user.table.actions.showRecords', 'Show records'), href: `/backend/entities/user/${encodeURIComponent(row.entityId)}/records` },
+            { label: t('common.edit', 'Edit'), href: `/backend/entities/user/${encodeURIComponent(row.entityId)}` },
+            { label: t('entities.user.table.actions.showRecords', 'Show records'), href: `/backend/entities/user/${encodeURIComponent(row.entityId)}/records` },
           ]}
         />
       )}

package/src/modules/feature_toggles/components/FeatureTogglesTable.tsx

@@ -205,11 +205,12 @@ export function FeatureTogglesTable() {
       }}
       rowActions={(row) => (
         <RowActions items={[
-          { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/feature-toggles/global/${row.id}/edit` },
-          { id: 'view', label: t('common.view', 'Overrides'), href: `/backend/feature-toggles/global/${row.id}` },
-          { id: 'delete', label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
+          { label: t('common.edit', 'Edit'), href: `/backend/feature-toggles/global/${row.id}/edit` },
+          { label: t('common.view', 'Overrides'), href: `/backend/feature-toggles/global/${row.id}` },
+          { label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
         ]} />
       )}
     />
   )
 }
+

package/src/modules/feature_toggles/components/OverridesTable.tsx

@@ -161,7 +161,7 @@ export default function OverridesTable() {
             }}
             rowActions={(row) => (
                 <RowActions items={[
-                    { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/feature-toggles/global/${row.toggleId}` },
+                    { label: t('common.edit', 'Edit'), href: `/backend/feature-toggles/global/${row.toggleId}` },
                 ]} />
             )}
             error={error ? error.message : undefined}

package/src/modules/planner/backend/planner/availability-rulesets/page.tsx

@@ -202,8 +202,8 @@ export default function PlannerAvailabilityRuleSetsPage() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: labels.actions.edit, href: `/backend/planner/availability-rulesets/${row.id}` },
-                { id: 'delete', label: labels.actions.delete, destructive: true, onSelect: () => { void handleDelete(row) } },
+                { label: labels.actions.edit, href: `/backend/planner/availability-rulesets/${row.id}` },
+                { label: labels.actions.delete, destructive: true, onSelect: () => { void handleDelete(row) } },
               ]}
             />
           )}

package/src/modules/query_index/components/QueryIndexesTable.tsx

@@ -366,15 +366,13 @@ export default function QueryIndexesTable() {
       onSortingChange={setSorting}
       perspective={{ tableId: 'query_index.status.list' }}
       rowActions={(row) => {
-        const items: Array<{ id: string; label: string; onSelect: () => void; destructive?: boolean }> = [
-          { id: 'reindex', label: t('query_index.table.actions.reindex'), onSelect: () => trigger('reindex', row.entityId) },
+        const items: Array<{ label: string; onSelect: () => void; destructive?: boolean }> = [
+          { label: t('query_index.table.actions.reindex'), onSelect: () => trigger('reindex', row.entityId) },
           {
-            id: 'reindex-force',
             label: t('query_index.table.actions.reindexForce'),
             onSelect: () => trigger('reindex', row.entityId, { force: true }),
           },
           {
-            id: 'purge',
             label: t('query_index.table.actions.purge'),
             destructive: true,
             onSelect: () => trigger('purge', row.entityId),
@@ -384,12 +382,10 @@ export default function QueryIndexesTable() {
         if (row.vectorEnabled) {
           items.push(
             {
-              id: 'vector-reindex',
               label: t('query_index.table.actions.vectorReindex'),
               onSelect: () => triggerVector('reindex', row.entityId),
             },
             {
-              id: 'vector-purge',
               label: t('query_index.table.actions.vectorPurge'),
               destructive: true,
               onSelect: () => triggerVector('purge', row.entityId),
@@ -400,12 +396,10 @@ export default function QueryIndexesTable() {
         if (row.fulltextEnabled) {
           items.push(
             {
-              id: 'fulltext-reindex',
               label: t('query_index.table.actions.fulltextReindex'),
               onSelect: () => triggerFulltext('reindex', row.entityId),
             },
             {
-              id: 'fulltext-purge',
               label: t('query_index.table.actions.fulltextPurge'),
               destructive: true,
               onSelect: () => triggerFulltext('purge', row.entityId),

package/src/modules/resources/backend/resources/resource-types/page.tsx

@@ -266,10 +266,10 @@ export default function ResourcesResourceTypesPage() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: translations.actions.edit, href: `/backend/resources/resource-types/${row.id}/edit` },
+                { label: translations.actions.edit, href: `/backend/resources/resource-types/${row.id}/edit` },
                 ...(row.resourceCount > 0
                   ? []
-                  : [{ id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => handleDelete(row) }]),
+                  : [{ label: translations.actions.delete, destructive: true, onSelect: () => handleDelete(row) }]),
               ]}
             />
           )}

package/src/modules/resources/backend/resources/resources/page.tsx

@@ -481,8 +481,8 @@ export default function ResourcesResourcesPage() {
             if (!canManage || row.rowKind !== 'resource') return null
             return (
               <RowActions items={[
-                { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/resources/resources/${encodeURIComponent(row.id)}` },
-                { id: 'delete', label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
+                { label: t('common.edit', 'Edit'), href: `/backend/resources/resources/${encodeURIComponent(row.id)}` },
+                { label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
               ]} />
             )
           }}

package/src/modules/sales/backend/sales/channels/offers/page.tsx

@@ -334,12 +334,10 @@ export default function SalesChannelOffersListPage() {
               <RowActions
                 items={[
                   {
-                    id: 'edit',
                     label: t('sales.channels.offers.actions.edit', 'Edit'),
                     href: `/backend/sales/channels/${row.channelId}/offers/${row.id}/edit`,
                   },
                   {
-                    id: 'delete',
                     label: t('sales.channels.offers.actions.delete', 'Delete'),
                     onSelect: () => handleDelete(row),
                     destructive: true,

package/src/modules/sales/backend/sales/channels/page.tsx

@@ -191,12 +191,10 @@ export default function SalesChannelsPage() {
             <RowActions
               items={[
                 {
-                  id: 'edit',
                   label: t('sales.channels.table.actions.edit', 'Edit'),
                   href: `/backend/sales/channels/${row.id}/edit`,
                 },
                 {
-                  id: 'delete',
                   label: t('sales.channels.table.actions.delete', 'Delete'),
                   onSelect: () => handleDelete(row),
                 },

package/src/modules/sales/commands/documents.ts

@@ -11,8 +11,6 @@ import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { deriveResourceFromCommandId, invalidateCrudCache } from '@open-mercato/shared/lib/crud/cache'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
-import { resolveNotificationService } from '../../notifications/lib/notificationService'
-import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
 import { setRecordCustomFields } from '@open-mercato/core/modules/entities/lib/helpers'
 import { loadCustomFieldValues } from '@open-mercato/shared/lib/crud/custom-fields'
 import { normalizeCustomFieldValues } from '@open-mercato/shared/lib/custom-fields/normalize'
@@ -90,7 +88,6 @@ import { resolveDictionaryEntryValue } from '../lib/dictionaries'
 import { resolveStatusEntryIdByValue } from '../lib/statusHelpers'
 import { SalesDocumentNumberGenerator } from '../services/salesDocumentNumberGenerator'
 import { loadSalesSettings } from './settings'
-import { notificationTypes } from '../notifications'
 
 type DocumentAddressSnapshot = {
   id: string
@@ -3114,37 +3111,6 @@ const createQuoteCommand: CommandHandler<QuoteCreateInput, { quoteId: string }>
     })
     await em.flush()
 
-    // Create notification for users with sales.quotes.manage feature
-    try {
-      const notificationService = resolveNotificationService(ctx.container)
-      const typeDef = notificationTypes.find((type) => type.type === 'sales.quote.created')
-      if (typeDef) {
-        const totalAmount = quote.grandTotalGrossAmount && quote.currencyCode
-          ? `${quote.grandTotalGrossAmount} ${quote.currencyCode}`
-          : ''
-        const totalDisplay = totalAmount ? ` (${totalAmount})` : ''
-        const notificationInput = buildFeatureNotificationFromType(typeDef, {
-          requiredFeature: 'sales.quotes.manage',
-          bodyVariables: {
-            quoteNumber: quote.quoteNumber,
-            total: totalDisplay,
-            totalAmount,
-          },
-          sourceEntityType: 'sales:quote',
-          sourceEntityId: quote.id,
-          linkHref: `/backend/sales/quotes/${quote.id}`,
-        })
-
-        await notificationService.createForFeature(notificationInput, {
-          tenantId: quote.tenantId,
-          organizationId: quote.organizationId ?? null,
-        })
-      }
-    } catch (err) {
-      // Notification creation is non-critical, don't fail the command
-      console.error('[sales.quotes.create] Failed to create notification:', err)
-    }
-
     return { quoteId: quote.id }
   },
   captureAfter: async (_input, result, ctx) => {
@@ -3816,37 +3782,6 @@ const createOrderCommand: CommandHandler<OrderCreateInput, { orderId: string }>
     })
     await em.flush()
 
-    // Create notification for users with sales.orders.manage feature
-    try {
-      const notificationService = resolveNotificationService(ctx.container)
-      const typeDef = notificationTypes.find((type) => type.type === 'sales.order.created')
-      if (typeDef) {
-        const totalAmount = order.grandTotalGrossAmount && order.currencyCode
-          ? `${order.grandTotalGrossAmount} ${order.currencyCode}`
-          : ''
-        const totalDisplay = totalAmount ? ` (${totalAmount})` : ''
-        const notificationInput = buildFeatureNotificationFromType(typeDef, {
-          requiredFeature: 'sales.orders.manage',
-          bodyVariables: {
-            orderNumber: order.orderNumber,
-            total: totalDisplay,
-            totalAmount,
-          },
-          sourceEntityType: 'sales:order',
-          sourceEntityId: order.id,
-          linkHref: `/backend/sales/orders/${order.id}`,
-        })
-
-        await notificationService.createForFeature(notificationInput, {
-          tenantId: order.tenantId,
-          organizationId: order.organizationId ?? null,
-        })
-      }
-    } catch (err) {
-      // Notification creation is non-critical, don't fail the command
-      console.error('[sales.orders.create] Failed to create notification:', err)
-    }
-
     return { orderId: order.id }
   },
   captureAfter: async (_input, result, ctx) => {

package/src/modules/sales/commands/payments.ts

@@ -35,9 +35,6 @@ import { invalidateCrudCache } from '@open-mercato/shared/lib/crud/cache'
 import { emitCrudSideEffects } from '@open-mercato/shared/lib/commands/helpers'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
-import { resolveNotificationService } from '../../notifications/lib/notificationService'
-import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
-import { notificationTypes } from '../notifications'
 
 export type PaymentAllocationSnapshot = {
   id: string
@@ -425,36 +422,6 @@ const createPaymentCommand: CommandHandler<
     const totals = await recomputeOrderPaymentTotals(em, order)
     await em.flush()
     await invalidateOrderCache(ctx.container, order, ctx.auth?.tenantId ?? null)
-
-    // Create notification for payment received
-    try {
-      const notificationService = resolveNotificationService(ctx.container)
-      const typeDef = notificationTypes.find((type) => type.type === 'sales.payment.received')
-      if (typeDef) {
-        const amountDisplay = payment.amount && payment.currencyCode
-          ? `${payment.currencyCode} ${payment.amount}`
-          : ''
-        const notificationInput = buildFeatureNotificationFromType(typeDef, {
-          requiredFeature: 'sales.orders.manage',
-          bodyVariables: {
-            orderNumber: order.orderNumber ?? '',
-            amount: amountDisplay,
-          },
-          sourceEntityType: 'sales:order',
-          sourceEntityId: order.id,
-          linkHref: `/backend/sales/orders/${order.id}`,
-        })
-
-        await notificationService.createForFeature(notificationInput, {
-          tenantId: payment.tenantId,
-          organizationId: payment.organizationId ?? null,
-        })
-      }
-    } catch (err) {
-      // Notification creation is non-critical, don't fail the command
-      console.error('[sales.payments.create] Failed to create notification:', err)
-    }
-
     return { paymentId: payment.id, orderTotals: totals }
   },
   captureAfter: async (_input, result, ctx) => {

package/src/modules/sales/components/AdjustmentKindSettings.tsx

@@ -333,8 +333,8 @@ export function AdjustmentKindSettings() {
           rowActions={(entry) => (
             <RowActions
               items={[
-                { id: 'edit', label: labels.edit, onSelect: () => openDialog({ mode: 'edit', entry }) },
-                { id: 'delete', label: labels.delete, destructive: true, onSelect: () => { void handleDelete(entry) } },
+                { label: labels.edit, onSelect: () => openDialog({ mode: 'edit', entry }) },
+                { label: labels.delete, destructive: true, onSelect: () => { void handleDelete(entry) } },
               ]}
             />
           )}

package/src/modules/sales/components/PaymentMethodsSettings.tsx

@@ -526,8 +526,8 @@ export function PaymentMethodsSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

package/src/modules/sales/components/ShippingMethodsSettings.tsx

@@ -820,8 +820,8 @@ export function ShippingMethodsSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

package/src/modules/sales/components/TaxRatesSettings.tsx

@@ -335,8 +335,8 @@ export function TaxRatesSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

package/src/modules/sales/components/channels/SalesChannelOffersPanel.tsx

@@ -185,12 +185,10 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
             <RowActions
               items={[
                 {
-                  id: 'edit',
                   label: t('sales.channels.offers.actions.edit', 'Edit'),
                   href: `/backend/sales/channels/${channelId}/offers/${row.id}/edit`,
                 },
                 {
-                  id: 'delete',
                   label: t('sales.channels.offers.actions.delete', 'Delete'),
                   onSelect: () => handleDelete(row),
                   destructive: true,

package/src/modules/sales/components/documents/AdjustmentsSection.tsx

@@ -427,12 +427,10 @@ export function SalesDocumentAdjustmentsSection({
       <RowActions
         items={[
           {
-            id: 'edit',
             label: t('ui.actions.edit', 'Edit'),
             onSelect: () => handleEdit(row),
           },
           {
-            id: 'delete',
             label: t('ui.actions.delete', 'Delete'),
             destructive: true,
             onSelect: () => handleDelete(row),

package/src/modules/sales/components/documents/PaymentsSection.tsx

@@ -295,9 +295,8 @@ export function SalesDocumentPaymentsSection({
           return (
             <RowActions
               items={[
-                { id: 'edit', label: editActionLabel, onSelect: () => openEditPayment(row.original) },
+                { label: editActionLabel, onSelect: () => openEditPayment(row.original) },
                 {
-                  id: 'delete',
                   label: deleteActionLabel,
                   destructive: true,
                   onSelect: () => void handleDelete(row.original),