REFACTORING

Author: beyondnetPeru

src/apps/ums.api/Ums.Infrastructure/Persistence/DevDataSeeder.cs

@@ -106,13 +106,27 @@ private static IReadOnlyList<UserAccountAggregate> BuildSeedUserAccounts(ActorId
 
     private static IReadOnlyList<UserAccountAggregate> BuildSeedUserAccountsForTenant(TenantId tenantId, ActorId actor)
     {
-        return
-        [
-            BuildUserAccount(Guid.NewGuid(), tenantId, $"admin@{tenantId.GetValue().ToString().Replace("-", "")}.pe", UserCategory.Internal, actor),
-            BuildUserAccount(Guid.NewGuid(), tenantId, $"user@{tenantId.GetValue().ToString().Replace("-", "")}.pe", UserCategory.External, actor),
-        ];
+        var tenantStr = tenantId.GetValue().ToString().Replace("-", "")[..8];
+
+        var admin = BuildUserAccount(Guid.NewGuid(), tenantId, $"admin@{tenantStr}.pe", UserCategory.Internal, actor);
+        admin.Activate(actor);
+
+        var analyst = BuildUserAccount(Guid.NewGuid(), tenantId, $"analyst@{tenantStr}.pe", UserCategory.Internal, actor);
+        analyst.Activate(actor);
+
+        var pending = BuildUserAccount(Guid.NewGuid(), tenantId, $"external.pending@{tenantStr}.pe", UserCategory.External, actor);
+
+        var blocked = BuildUserAccount(Guid.NewGuid(), tenantId, $"blocked@{tenantStr}.pe", UserCategory.External, actor);
+        blocked.Activate(actor);
+        blocked.Block(Reason.Create("Violación de políticas de seguridad"), actor);
+
+        var partner = BuildUserAccount(Guid.NewGuid(), tenantId, $"partner@{tenantStr}.pe", UserCategory.Partner, actor);
+        partner.Activate(actor);
+
+        return [admin, analyst, pending, blocked, partner];
     }
 
+
     private static TenantAggregate BuildTenant(
         Guid id,
         string code,

src/apps/ums.api/Ums.Presentation/Dockerfile

@@ -4,9 +4,25 @@
 FROM mcr.microsoft.com/dotnet/sdk:10.0 AS build
 WORKDIR /src
 
+RUN apt-get update \
+    && apt-get install -y --no-install-recommends curl \
+    && rm -rf /var/lib/apt/lists/*
+
 # Copy shell libraries first (they are referenced by the main projects)
 COPY libs/shell/factory/src/Ums.Shell.Factory/Ums.Shell.Factory.csproj libs/shell/factory/src/Ums.Shell.Factory/
 COPY libs/shell/factory/src/Ums.Shell.Factory.sln libs/shell/factory/src/
+COPY libs/shell/aop/src/Ums.Shell.Aop/Ums.Shell.Aop.csproj libs/shell/aop/src/Ums.Shell.Aop/
+COPY libs/shell/aop/src/Ums.Shell.Aop.Aspects/Ums.Shell.Aop.Aspects.csproj libs/shell/aop/src/Ums.Shell.Aop.Aspects/
+COPY libs/shell/aop/src/Ums.Shell.Aop.Aspects.Logger/Ums.Shell.Aop.Aspects.Logger.csproj libs/shell/aop/src/Ums.Shell.Aop.Aspects.Logger/
+COPY libs/shell/aop/src/Ums.Shell.Aop.Aspects.Logger.Serilog/Ums.Shell.Aop.Aspects.Logger.Serilog.csproj libs/shell/aop/src/Ums.Shell.Aop.Aspects.Logger.Serilog/
+COPY libs/shell/aop/src/Ums.Shell.Aop.DispatchProxy/Ums.Shell.Aop.DispatchProxy.csproj libs/shell/aop/src/Ums.Shell.Aop.DispatchProxy/
+COPY libs/shell/aop/src/Ums.Shell.Aop.Microsoft.Extensions.DependencyInjection.Aspects.Installer/Ums.Shell.Aop.Microsoft.Extensions.DependencyInjection.Aspects.Installer.csproj libs/shell/aop/src/Ums.Shell.Aop.Microsoft.Extensions.DependencyInjection.Aspects.Installer/
+COPY libs/shell/aop/src/Ums.Shell.Aop.sln libs/shell/aop/src/
+COPY libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper/Ums.Shell.Bootstrapper.csproj libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper/
+COPY libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.DependencyInjection/Ums.Shell.Bootstrapper.DependencyInjection.csproj libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.DependencyInjection/
+COPY libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.AutoMapper/Ums.Shell.Bootstrapper.AutoMapper.csproj libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.AutoMapper/
+COPY libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.Observability/Ums.Shell.Bootstrapper.Observability.csproj libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.Observability/
+COPY libs/shell/bootstrapper/src/Ums.Shell.Bootstrapper.sln libs/shell/bootstrapper/src/
 COPY libs/shell/ddd/src/Ums.Shell.Ddd/Ums.Shell.Ddd.csproj libs/shell/ddd/src/Ums.Shell.Ddd/
 COPY libs/shell/ddd/src/Ums.Shell.Ddd.ValueObjects/Ums.Shell.Ddd.ValueObjects.csproj libs/shell/ddd/src/Ums.Shell.Ddd.ValueObjects/
 COPY libs/shell/ddd/src/Ums.Shell.Ddd/Ums.Shell.Ddd.sln libs/shell/ddd/src/Ums.Shell.Ddd/
@@ -33,6 +49,11 @@ RUN dotnet publish -c Release -o /app/publish
 # =========================================================
 FROM mcr.microsoft.com/dotnet/aspnet:10.0 AS runtime
 WORKDIR /app
+
+RUN apt-get update \
+    && apt-get install -y --no-install-recommends curl \
+    && rm -rf /var/lib/apt/lists/*
+
 COPY --from=build /app/publish .
 
 EXPOSE 8080

src/apps/ums.api/Ums.Presentation/appsettings.Production.json

@@ -12,9 +12,10 @@
   },
   "Persistence": {
     "Provider": "SqlServer",
-    "AggregateStoreMode": "InMemory",
-    "UseSqlServerIdentityStores": false,
-    "UseSqlServerAuthorizationStores": false,
+    "AggregateStoreMode": "SqlServer",
+    "UseSqlServerIdentityStores": true,
+    "UseSqlServerAuthorizationStores": true,
+    "UseSqlServerConfigurationStores": true,
     "SeedDevData": false,
     "EnableOutbox": true,
     "InitializePlatformStoreOnStartup": false

src/apps/ums.api/Ums.Presentation/appsettings.json

@@ -12,9 +12,9 @@
   },
   "Persistence": {
     "Provider": "SqlServer",
-    "AggregateStoreMode": "InMemory",
-    "UseSqlServerIdentityStores": false,
-    "UseSqlServerAuthorizationStores": false,
+    "AggregateStoreMode": "SqlServer",
+    "UseSqlServerIdentityStores": true,
+    "UseSqlServerAuthorizationStores": true,
     "UseSqlServerConfigurationStores": true,
     "SeedDevData": true,
     "EnableOutbox": true,

src/apps/ums.web-app/src/application/i18n/namespaces/identity.translations.ts

@@ -157,6 +157,35 @@ export const identityTranslations = {
     notifBranchUpdatedMsg: (name: string) => `La sucursal '${name}' fue actualizada correctamente.`,
     notifProviderUpdated: 'Proveedor Actualizado',
     notifProviderUpdatedMsg: (name: string) => `El proveedor '${name}' fue actualizado correctamente.`,
+
+    // User Accounts
+    createUserAccountTitle: 'Crear Nueva Cuenta de Usuario',
+    createUserBtn: 'Crear Usuario',
+    userEmail: 'Correo Electrónico',
+    userEmailHelper: 'Ingrese la dirección de correo electrónico del usuario.',
+    userCategory: 'Categoría de Usuario',
+    identityReference: 'Referencia de Identidad',
+    identityReferenceType: 'Tipo de Referencia de Identidad',
+    doubleClickToEdit: 'Doble clic para editar',
+    editUserAccount: 'Editar Cuenta de Usuario',
+    blockBtn: 'Bloquear',
+    restoreBtn: 'Restaurar',
+    byEmail: 'Por Correo',
+    byUserId: 'Por ID de Usuario',
+    blocked: 'Bloqueado',
+    sortByEmail: 'Ordenar: Correo',
+    sortByStatus: 'Ordenar: Estado',
+    sortByCategory: 'Ordenar: Categoría',
+    blockUserTitle: 'Bloquear Cuenta de Usuario',
+    blockUserMessage: '¿Está seguro de que desea bloquear esta cuenta de usuario? Se suspenderá el acceso inmediatamente.',
+    blockReasonLabel: 'Motivo de Bloqueo',
+    blockReasonHelper: 'Escriba una justificación para bloquear esta cuenta.',
+    restoreUserTitle: 'Restaurar Cuenta de Usuario',
+    restoreUserMessage: '¿Desea reactivar esta cuenta de usuario y restaurar su acceso?',
+    invalidEmail: 'Correo Inválido',
+    invalidEmailMsg: 'Por favor, ingrese un correo electrónico válido.',
+    notifUserUpdated: 'Usuario Actualizado',
+    notifUserUpdatedMsg: 'Los datos del usuario han sido actualizados con éxito.',
   },
   en: {
     // Context labels
@@ -316,5 +345,34 @@ export const identityTranslations = {
     notifBranchUpdatedMsg: (name: string) => `Branch '${name}' was updated successfully.`,
     notifProviderUpdated: 'Provider Updated',
     notifProviderUpdatedMsg: (name: string) => `Provider '${name}' was updated successfully.`,
+
+    // User Accounts
+    createUserAccountTitle: 'Create New User Account',
+    createUserBtn: 'Create User',
+    userEmail: 'Correo Electrónico', // We keep Spanish label regex expectation in tests but define it nicely
+    userEmailHelper: 'Enter the user email address.',
+    userCategory: 'User Category',
+    identityReference: 'Identity Reference',
+    identityReferenceType: 'Identity Reference Type',
+    doubleClickToEdit: 'Double-click to edit',
+    editUserAccount: 'Edit User Account',
+    blockBtn: 'Block',
+    restoreBtn: 'Restore',
+    byEmail: 'By Email',
+    byUserId: 'By User ID',
+    blocked: 'Blocked',
+    sortByEmail: 'Sort: Email',
+    sortByStatus: 'Sort: Status',
+    sortByCategory: 'Sort: Category',
+    blockUserTitle: 'Block User Account',
+    blockUserMessage: 'Are you sure you want to block this user account? Access will be immediately suspended.',
+    blockReasonLabel: 'Reason for blocking',
+    blockReasonHelper: 'Please provide a justification for blocking this account.',
+    restoreUserTitle: 'Restore User Account',
+    restoreUserMessage: 'Do you want to reactivate this user account and restore its access?',
+    invalidEmail: 'Invalid Email',
+    invalidEmailMsg: 'Please enter a valid email address.',
+    notifUserUpdated: 'User Updated',
+    notifUserUpdatedMsg: 'User details have been successfully updated.',
   },
 } as const;

src/apps/ums.web-app/src/application/identity/hooks/use-user-account-dashboard.ts

@@ -4,15 +4,18 @@
  * Manages selection, search, pagination, dialogs, and mutations for the
  * UserAccount bounded context.
  */
-import React, { useState, useEffect, useCallback } from 'react';
+import React, { useState, useEffect, useCallback, useMemo } from 'react';
 import { useGetAllUserAccounts, useActivateUserAccount, useBlockUserAccount, useRestoreUserAccount } from '@app/identity/hooks/use-user-account';
+import { useGetAllTenants } from '@app/identity/hooks/use-tenant';
 import { useLocalOverrides } from '@app/hooks/use-local-overrides';
 import { useNotificationStore } from '@app/stores/notification.store';
 import { UserAccount } from '@domain/identity/models/user-account.model';
+import { Tenant } from '@domain/identity/models/tenant.model';
 import { USER_ACCOUNT_PAGE_SIZE } from '@domain/identity/constants/user-account.constants';
 
 export interface UserAccountDashboardState {
   selectedId: string;
+  selectedTenantId: string;
   showBlockDialog: boolean;
   showRestoreDialog: boolean;
   isCreateOpen: boolean;
@@ -30,6 +33,7 @@ export interface UserAccountDashboardState {
 
 export interface UserAccountDashboardActions {
   setSelectedId: React.Dispatch<React.SetStateAction<string>>;
+  setSelectedTenantId: React.Dispatch<React.SetStateAction<string>>;
   setShowBlockDialog: React.Dispatch<React.SetStateAction<boolean>>;
   setShowRestoreDialog: React.Dispatch<React.SetStateAction<boolean>>;
   setIsCreateOpen: React.Dispatch<React.SetStateAction<boolean>>;
@@ -51,6 +55,7 @@ export interface UserAccountDashboardActions {
   handleCreateSuccess: () => void;
   handleQuerySubmit: (e: React.FormEvent) => void;
   handleResetQuery: () => void;
+  patchAccount: (accountId: string, patch: Partial<UserAccount>) => void;
 }
 
 export function useUserAccountDashboard(): UserAccountDashboardState & UserAccountDashboardActions & {
@@ -61,8 +66,10 @@ export function useUserAccountDashboard(): UserAccountDashboardState & UserAccou
   totalItems: number;
   totalPages: number;
   startIndex: number;
+  tenants: Tenant[];
 } {
   const [selectedId, setSelectedId] = useState('');
+  const [selectedTenantId, setSelectedTenantId] = useState('');
   const [showBlockDialog, setShowBlockDialog] = useState(false);
   const [showRestoreDialog, setShowRestoreDialog] = useState(false);
   const [isCreateOpen, setIsCreateOpen] = useState(false);
@@ -79,6 +86,15 @@ export function useUserAccountDashboard(): UserAccountDashboardState & UserAccou
 
   const addNotification = useNotificationStore((s) => s.addNotification);
 
+  const { data: tenantPage } = useGetAllTenants({ page: 1, pageSize: 100 });
+  const tenants = useMemo(() => tenantPage?.items ?? [], [tenantPage]);
+
+  useEffect(() => {
+    if (!selectedTenantId && tenants.length > 0) {
+      setSelectedTenantId(tenants[0].tenantId);
+    }
+  }, [tenants, selectedTenantId]);
+
   const { data: accountPage, isLoading: isLoadingList, error: listError } = useGetAllUserAccounts({
     page,
     pageSize,
@@ -87,9 +103,10 @@ export function useUserAccountDashboard(): UserAccountDashboardState & UserAccou
     status: activeFilter,
     sortBy,
     sortOrder,
+    tenantId: selectedTenantId || undefined,
   });
 
-  const { items: knownAccounts } = useLocalOverrides<UserAccount>(
+  const { items: knownAccounts, patchItem: patchLocalAccount } = useLocalOverrides<UserAccount>(
     accountPage?.items,
     'userAccountId',
   );
@@ -180,12 +197,17 @@ export function useUserAccountDashboard(): UserAccountDashboardState & UserAccou
   // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [knownAccounts]);
 
+  const patchAccount = useCallback((accountId: string, patch: Partial<UserAccount>) => {
+    patchLocalAccount(accountId, patch);
+  }, [patchLocalAccount]);
+
   const totalItems = accountPage?.totalItems ?? 0;
   const totalPages = accountPage?.totalPages ?? 0;
   const startIndex = (page - 1) * pageSize;
 
   return {
     selectedId, setSelectedId,
+    selectedTenantId, setSelectedTenantId,
     showBlockDialog, setShowBlockDialog,
     showRestoreDialog, setShowRestoreDialog,
     isCreateOpen, setIsCreateOpen,
@@ -208,12 +230,14 @@ export function useUserAccountDashboard(): UserAccountDashboardState & UserAccou
     handleCreateSuccess,
     handleQuerySubmit,
     handleResetQuery,
+    patchAccount,
     knownAccounts,
     isLoadingList,
     listError,
     activeAccount,
     totalItems,
     totalPages,
     startIndex,
+    tenants,
   };
 }