fix(resource-adm): Fix misleading "Sist endret" field in resources table

backend/src/Designer/Services/Implementation/GiteaAPIWrapper/GiteaAPIWrapper.cs

@@ -245,18 +245,6 @@ public async Task<ListviewServiceResource> MapServiceResourceToListViewResource(
                 }
             }
 
-            if (string.IsNullOrEmpty(listviewResource.CreatedBy))
-            {
-                string localUserName = AuthenticationHelper.GetDeveloperUserName(_httpContextAccessor.HttpContext);
-                string userFullName = await GetCachedUserFullName(localUserName);
-                listviewResource.CreatedBy = userFullName;
-            }
-
-            if (listviewResource.LastChanged == null)
-            {
-                listviewResource.LastChanged = DateTime.Now;
-            }
-
             return listviewResource;
         }
 

frontend/language/src/nb.json

@@ -101,7 +101,7 @@
   "app_deployment.pipeline_deployment.build_result.partiallySucceeded": "Versjonen er publisert i miljøet, men har noen feil eller mangler.",
   "app_deployment.pipeline_deployment.build_result.succeeded": "Versjonen er publisert i miljøet.",
   "app_deployment.pipeline_undeploy.build_result.failed": "Kunne ikke avpublisere appen. Prøv på nytt nå, eller vent litt og prøv igjen.",
-  "app_deployment.pipeline_undeploy.build_result.none": "Avpubliserer\u2026",
+  "app_deployment.pipeline_undeploy.build_result.none": "Avpubliserer…",
   "app_deployment.pipeline_undeploy.build_result.succeeded": "Appen er tatt bort fra miljøet",
   "app_deployment.private_app_owner": "Som privatperson kan du ikke publisere denne appen til noe miljø.",
   "app_deployment.private_app_owner_help": "Det er bare brukere i en organisasjon som kan publisere denne appen til et miljø. <a>Ta kontakt med oss</a> hvis du trenger hjelp.",
@@ -249,7 +249,7 @@
   "dashboard.resource_status_label": "Driftsmeldinger",
   "dashboard.resource_table_header_createdby": "Opprettet av",
   "dashboard.resource_table_header_environment": "Miljø",
-  "dashboard.resource_table_header_last_changed": "Sist endret",
+  "dashboard.resource_table_header_last_changed": "Siste delte endringer",
   "dashboard.resource_table_header_name": "Navn",
   "dashboard.resource_table_header_resourceid": "Ressurs-id",
   "dashboard.resource_table_no_resources_result": "Ingen ressurser funnet",

frontend/resourceadm/components/ResourceTable/ResourceTable.test.tsx

@@ -5,6 +5,7 @@ import { ResourceTable } from './ResourceTable';
 import type { ResourceListItem } from 'app-shared/types/ResourceAdm';
 import { textMock } from '@studio/testing/mocks/i18nMock';
 import userEvent from '@testing-library/user-event';
+import { LOCAL_RESOURCE_CHANGED_TIME } from 'resourceadm/utils/resourceListUtils';
 
 const resource1Title = 'tittel 1';
 const mockResourceListItem1: ResourceListItem = {
@@ -25,15 +26,24 @@ const mockResourceListItem2: ResourceListItem = {
 const resource3Title = 'tittel 3';
 const mockResourceListItem3: ResourceListItem = {
   title: { nb: resource3Title, en: '', nn: '' },
-  createdBy: 'John Doe',
+  createdBy: '',
   lastChanged: null,
   identifier: 'resource-3',
   environments: ['at22'],
 };
+const resource4Title = 'tittel 4';
+const mockResourceListItem4: ResourceListItem = {
+  title: { nb: resource4Title, en: '', nn: '' },
+  createdBy: '',
+  lastChanged: LOCAL_RESOURCE_CHANGED_TIME,
+  identifier: 'resource-4',
+  environments: ['gitea'],
+};
 const mockResourceList: ResourceListItem[] = [
   mockResourceListItem1,
   mockResourceListItem2,
   mockResourceListItem3,
+  mockResourceListItem4,
 ];
 
 describe('ResourceTable', () => {
@@ -98,6 +108,13 @@ describe('ResourceTable', () => {
     expect(lastChangedCell).toBeInTheDocument();
   });
 
+  it('displays last changed date blank if resource has last changed date in the future', () => {
+    render(<ResourceTable {...defaultProps} list={[mockResourceListItem4]} />);
+
+    const lastChangedCell = screen.queryByText('31.12.9999');
+    expect(lastChangedCell).not.toBeInTheDocument();
+  });
+
   it('displays environments for resource', () => {
     render(<ResourceTable {...defaultProps} />);
 

frontend/resourceadm/components/ResourceTable/ResourceTable.tsx

@@ -6,6 +6,7 @@ import { StudioButton, StudioSpinner, StudioTableLocalPagination } from '@studio
 import type { Columns } from '@studio/components';
 import type { ResourceListItem } from 'app-shared/types/ResourceAdm';
 import { useTranslation } from 'react-i18next';
+import { LOCAL_RESOURCE_CHANGED_TIME } from '../../utils/resourceListUtils';
 
 export type ResourceTableProps = {
   /**
@@ -145,14 +146,16 @@ export const ResourceTable = ({
       accessor: 'lastChanged',
       heading: t('dashboard.resource_table_header_last_changed'),
       sortable: true,
-      bodyCellFormatter: (value: string) =>
-        value
-          ? new Date(value).toLocaleDateString('no-NB', {
-              year: 'numeric',
-              month: '2-digit',
-              day: '2-digit',
-            })
-          : '',
+      bodyCellFormatter: (value: string) => {
+        if (!value || new Date(value).getTime() === LOCAL_RESOURCE_CHANGED_TIME.getTime()) {
+          return '';
+        }
+        return new Date(value).toLocaleDateString('no-NB', {
+          year: 'numeric',
+          month: '2-digit',
+          day: '2-digit',
+        });
+      },
     },
     {
       accessor: 'environments',

frontend/resourceadm/hooks/queries/useGetResourceListQuery.ts

@@ -3,7 +3,7 @@ import { useQuery } from '@tanstack/react-query';
 import { useServicesContext } from 'app-shared/contexts/ServicesContext';
 import { QueryKey } from 'app-shared/types/QueryKey';
 import type { ResourceListItem } from 'app-shared/types/ResourceAdm';
-import { sortResourceListByDate } from '../../utils/mapperUtils';
+import { setLastChangedAndSortResourceListByDate } from '../../utils/mapperUtils';
 
 /**
  * Query to get the list of resources. It maps the date to correct display format
@@ -23,7 +23,7 @@ export const useGetResourceListQuery = (
     queryKey: [QueryKey.ResourceList, org],
     queryFn: () => getResourceList(org),
     select: (resourceListItems: ResourceListItem[]) =>
-      resourceListItems && sortResourceListByDate(resourceListItems),
+      resourceListItems && setLastChangedAndSortResourceListByDate(resourceListItems),
     enabled: !disabled,
   });
 };

frontend/resourceadm/utils/mapperUtils/index.ts

@@ -1,5 +1,5 @@
 export {
-  sortResourceListByDate,
+  setLastChangedAndSortResourceListByDate,
   mapAltinn2LinkServiceToSelectOption,
   mapSelectOptiontoAltinn2LinkService,
 } from './mapperUtils';

frontend/resourceadm/utils/mapperUtils/mapperUtils.test.ts

@@ -1,5 +1,9 @@
 import type { Altinn2LinkService } from 'app-shared/types/Altinn2LinkService';
-import { mapAltinn2LinkServiceToSelectOption } from './mapperUtils';
+import {
+  mapAltinn2LinkServiceToSelectOption,
+  setLastChangedAndSortResourceListByDate,
+} from './mapperUtils';
+import { LOCAL_RESOURCE_CHANGED_TIME } from '../resourceListUtils';
 
 describe('mapperUtils', () => {
   describe('mapAltinn2LinkServiceToSelectOption', () => {
@@ -26,4 +30,67 @@ describe('mapperUtils', () => {
       expect(result[0].label).toBe('name1 (code1/edition1)');
     });
   });
+
+  describe('setLastChangedAndSortResourceListByDate', () => {
+    it('should sort the list by date', () => {
+      const resource1Id = 'resource-1';
+      const resource2Id = 'resource-2';
+      const resource3Id = 'resource-3';
+      const loadedResourceList = [
+        {
+          title: { nb: resource1Id, en: '', nn: '' },
+          createdBy: '',
+          lastChanged: null,
+          identifier: resource1Id,
+          environments: ['tt02'],
+        },
+        {
+          title: { nb: resource2Id, en: '', nn: '' },
+          createdBy: '',
+          lastChanged: null,
+          identifier: resource2Id,
+          environments: ['gitea'],
+        },
+        {
+          title: { nb: resource3Id, en: '', nn: '' },
+          createdBy: 'ulrik user',
+          lastChanged: new Date('2023-08-29'),
+          identifier: resource3Id,
+          environments: ['gitea', 'tt02'],
+        },
+      ];
+      const resultResourceList = setLastChangedAndSortResourceListByDate(loadedResourceList);
+      expect(resultResourceList[0].identifier).toBe(resource2Id);
+      expect(resultResourceList[1].identifier).toBe(resource3Id);
+      expect(resultResourceList[2].identifier).toBe(resource1Id);
+    });
+
+    it('should set lastChanged to static value if lastChanged is null and environments includes gitea', () => {
+      const loadedResourceList = [
+        {
+          title: { nb: 'resource-1', en: '', nn: '' },
+          createdBy: '',
+          lastChanged: null,
+          identifier: 'resource-1',
+          environments: ['gitea'],
+        },
+      ];
+      const resultResourceList = setLastChangedAndSortResourceListByDate(loadedResourceList);
+      expect(resultResourceList[0].lastChanged).toBe(LOCAL_RESOURCE_CHANGED_TIME);
+    });
+
+    it('should sort environments', () => {
+      const loadedResourceList = [
+        {
+          title: { nb: 'resource-1', en: '', nn: '' },
+          createdBy: '',
+          lastChanged: null,
+          identifier: 'resource-1',
+          environments: ['gitea', 'at24', 'at22', 'prod', 'tt02'],
+        },
+      ];
+      const resultResourceList = setLastChangedAndSortResourceListByDate(loadedResourceList);
+      expect(resultResourceList[0].environments).toEqual(['prod', 'tt02', 'at22', 'at24', 'gitea']);
+    });
+  });
 });

frontend/resourceadm/utils/mapperUtils/mapperUtils.ts

@@ -1,5 +1,6 @@
 import type { Altinn2LinkService } from 'app-shared/types/Altinn2LinkService';
 import type { ResourceListItem } from 'app-shared/types/ResourceAdm';
+import { LOCAL_RESOURCE_CHANGED_TIME } from '../../utils/resourceListUtils';
 
 const EnvOrder = ['prod', 'tt02', 'at22', 'at23', 'at24', 'gitea'];
 /**
@@ -9,10 +10,16 @@ const EnvOrder = ['prod', 'tt02', 'at22', 'at23', 'at24', 'gitea'];
  *
  * @returns the sorted list
  */
-export const sortResourceListByDate = (resourceList: ResourceListItem[]): ResourceListItem[] => {
+export const setLastChangedAndSortResourceListByDate = (
+  resourceList: ResourceListItem[],
+): ResourceListItem[] => {
   const listWithSortedEnvs = resourceList.map((resource) => {
     return {
       ...resource,
+      lastChanged:
+        resource.lastChanged === null && resource.environments.includes('gitea')
+          ? LOCAL_RESOURCE_CHANGED_TIME
+          : resource.lastChanged,
       environments: resource.environments.sort((a, b) => EnvOrder.indexOf(a) - EnvOrder.indexOf(b)),
     };
   });

frontend/resourceadm/utils/resourceListUtils/index.ts

@@ -1 +1 @@
-export { filterTableData } from './resourceListUtils';
+export { filterTableData, LOCAL_RESOURCE_CHANGED_TIME } from './resourceListUtils';

frontend/resourceadm/utils/resourceListUtils/resourceListUtils.ts

@@ -1,5 +1,7 @@
 import type { ResourceListItem } from 'app-shared/types/ResourceAdm';
 
+export const LOCAL_RESOURCE_CHANGED_TIME = new Date('9999-12-31');
+
 /**
  * Filter the list based on what is typed in the search box
  */