[UI] EVEREST-1175 Fix minor UI issues with texts (#460)

* fix: update texts * fix: tests --------- Co-authored-by: Fábio Silva <[email protected]>
percona · Jun 28, 2024 · a90a1bb · a90a1bb
1 parent 4410e73
commit a90a1bb
Show file tree

Hide file tree

Showing 10 changed files with 14 additions and 15 deletions.
diff --git a/ui/apps/everest/.e2e/db-cluster/db-cluster-overview.e2e.ts b/ui/apps/everest/.e2e/db-cluster/db-cluster-overview.e2e.ts
@@ -42,7 +42,7 @@ test.describe('DB Cluster Overview', async () => {
 
     await expect(page.getByText('Type: MySQL')).toBeVisible();
     await expect(page.getByText(`Name: ${dbClusterName}`)).toBeVisible();
-    await expect(page.getByText('Number of nodes: 1')).toBeVisible();
+    await expect(page.getByText('Nº nodes: 1')).toBeVisible();
     await expect(
       page.getByText('CPU: 600m').or(page.getByText('CPU: 0.6'))
     ).toBeVisible();

diff --git a/ui/apps/everest/.e2e/db-cluster/db-wizard/create-db-cluster/create-db-cluster.e2e.ts b/ui/apps/everest/.e2e/db-cluster/db-wizard/create-db-cluster/create-db-cluster.e2e.ts
@@ -138,7 +138,7 @@ test.describe('DB Cluster creation', () => {
     dbName = await page.getByTestId('text-input-db-name').inputValue();
 
     await moveForward(page);
-    await expect(page.getByText('Number of nodes: 3')).toBeVisible();
+    await expect(page.getByText('Nº nodes: 3')).toBeVisible();
 
     await resourcesStepCheck(page);
     await moveForward(page);
@@ -160,15 +160,15 @@ test.describe('DB Cluster creation', () => {
       dbName
     );
     await page.getByTestId('postgresql-toggle-button').click();
-    await expect(page.getByText('Number of nodes: 2')).toBeVisible();
+    await expect(page.getByText('Nº nodes: 2')).toBeVisible();
     // Now we change the number of nodes
     await page.getByTestId('button-edit-preview-resources').click();
     await page.getByTestId('toggle-button-nodes-3').click();
     await page.getByTestId('toggle-button-nodes-2').click();
     await page.getByTestId('button-edit-preview-basic-information').click();
     // Because 2 nodes is not valid for MongoDB, the default will be picked
     await page.getByTestId('mongodb-toggle-button').click();
-    await expect(page.getByText('Number of nodes: 3')).toBeVisible();
+    await expect(page.getByText('Nº nodes: 3')).toBeVisible();
     await page.getByTestId('button-edit-preview-backups').click();
 
     await expect(page.getByTestId('radio-option-logical')).not.toBeVisible();

diff --git a/...est/src/pages/database-form/database-form-body/steps/resources/resources-step.messages.ts b/...est/src/pages/database-form/database-form-body/steps/resources/resources-step.messages.ts
@@ -18,7 +18,7 @@ export const Messages = {
   pageDescription:
     'Configure the resources your new database will have access to.',
   labels: {
-    numberOfNodes: 'Number of nodes',
+    numberOfNodes: 'Nº nodes',
     resourceSizePerNode: 'Resource size per node',
     cpu: 'cpu',
     memory: 'memory',

diff --git a/ui/apps/everest/src/pages/database-form/database-preview/database-preview.test.tsx b/ui/apps/everest/src/pages/database-form/database-preview/database-preview.test.tsx
@@ -60,7 +60,7 @@ describe('DatabasePreview', () => {
     expect(screen.getByText('Type: MySQL')).toBeInTheDocument();
     expect(screen.getByText('Version: 1.0.0')).toBeInTheDocument();
 
-    expect(screen.queryByText('Number of nodes: 1')).not.toBeInTheDocument();
+    expect(screen.queryByText('Nº nodes: 1')).not.toBeInTheDocument();
   });
 
   it('should show values from previous steps', () => {
@@ -83,7 +83,7 @@ describe('DatabasePreview', () => {
     expect(screen.getByText('Type: MySQL')).toBeInTheDocument();
     expect(screen.getByText('Version: 1.0.0')).toBeInTheDocument();
 
-    expect(screen.getByText('Number of nodes: 1')).toBeInTheDocument();
+    expect(screen.getByText('Nº nodes: 1')).toBeInTheDocument();
     expect(screen.getByText('CPU: 1 CPU')).toBeInTheDocument();
     expect(screen.getByText('Disk: 30 GB')).toBeInTheDocument();
   });

diff --git a/ui/apps/everest/src/pages/database-form/database-preview/sections/resources-section.tsx b/ui/apps/everest/src/pages/database-form/database-preview/sections/resources-section.tsx
@@ -13,7 +13,7 @@ export const ResourcesPreviewSection = ({
 
   return (
     <>
-      <PreviewContentText text={`Number of nodes: ${numberOfNodes}`} />
+      <PreviewContentText text={`Nº nodes: ${numberOfNodes}`} />
       <PreviewContentText
         text={`CPU: ${Number.isNaN(parsedCPU) ? '' : `${parsedCPU} CPU`}`}
       />

diff --git a/ui/apps/everest/src/pages/databases/DbClusterView.tsx b/ui/apps/everest/src/pages/databases/DbClusterView.tsx
@@ -145,12 +145,12 @@ export const DbClusterView = () => {
       {
         accessorKey: 'nodes',
         id: 'nodes',
-        header: 'Number of nodes',
+        header: 'Nº nodes',
       },
       {
         accessorKey: 'namespace',
         id: 'namespace',
-        header: 'Namespaces',
+        header: 'Namespace',
       },
       {
         accessorKey: 'monitoringConfigName',

diff --git a/ui/apps/everest/src/pages/databases/dbClusterView.messages.tsx b/ui/apps/everest/src/pages/databases/dbClusterView.messages.tsx
@@ -45,7 +45,7 @@ export const Messages = {
     cpu: 'CPU',
     memory: 'Memory',
     disk: 'Disk',
-    nodes: 'Number of nodes',
+    nodes: 'Nº nodes',
     externalAccess: 'External Access',
     monitoring: 'Monitoring',
     enabled: 'Enabled',

diff --git a/ui/apps/everest/src/pages/db-cluster-details/cluster-overview/cluster-overview.messages.ts b/ui/apps/everest/src/pages/db-cluster-details/cluster-overview/cluster-overview.messages.ts
@@ -19,8 +19,7 @@ export const Messages = {
     name: (clusterName?: string) => `Name: ${clusterName}`,
     namespace: (namespace?: string) => `Namespace: ${namespace}`,
     version: (type?: string) => `Version: ${type}`,
-    numberOfNodes: (numberOfNodes?: number) =>
-      `Number of nodes: ${numberOfNodes}`,
+    numberOfNodes: (numberOfNodes?: number) => `Nº nodes: ${numberOfNodes}`,
     cpu: (cpu?: string | number) => `CPU: ${cpu}`,
     memory: (memory: string | number) => `Memory: ${memory}`,
     disk: (disk: string | number) => `Disk: ${disk}`,

diff --git a/ui/apps/everest/src/pages/settings/monitoring-endpoints/monitoring-endpoints.messages.tsx b/ui/apps/everest/src/pages/settings/monitoring-endpoints/monitoring-endpoints.messages.tsx
@@ -27,7 +27,7 @@ export const Messages = {
       'Percona Everest does not store PMM credentials, so fill in both the User and Password fields.',
   },
   fieldPlaceholders: {
-    name: 'Enter Monitoring Endpoint name',
+    name: 'Insert Monitoring endpoint display name',
     namespaces: 'Select namespaces',
     endpoint: 'Enter endpoint URL',
     user: 'Enter username',

diff --git a/ui/apps/everest/src/pages/settings/storage-locations/storage-locations.messages.tsx b/ui/apps/everest/src/pages/settings/storage-locations/storage-locations.messages.tsx
@@ -9,7 +9,7 @@ export const Messages = {
       description: 'Enter an optional description',
       namespaces: 'Select namespaces',
       type: 'Enter bucket name',
-      region: 'Enter regions',
+      region: 'Enter region',
       url: 'Enter URL',
       accessKey: 'Enter Access key',
       secretKey: 'Enter Secret key',