Fix bug with issue count not showing on overview page

During the migration of middleware, we somehow managed to cause this feature to be lost. #500
digital-land · Oct 8, 2024 · 9bf692f · 9bf692f
1 parent a462584
commit 9bf692f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/middleware/datasetOverview.middleware.js b/src/middleware/datasetOverview.middleware.js
@@ -45,7 +45,7 @@ const fetchEntityCount = fetchOne({
 })
 
 export const prepareDatasetOverviewTemplateParams = (req, res, next) => {
-  const { orgInfo, specification, columnSummary, entityCount, sources, dataset } = req
+  const { orgInfo, specification, columnSummary, entityCount, sources, dataset, issues } = req
 
   const mappingFields = columnSummary[0].mapping_field?.split(';') ?? []
   const nonMappingFields = columnSummary[0].non_mapping_field?.split(';') ?? []
@@ -89,6 +89,7 @@ export const prepareDatasetOverviewTemplateParams = (req, res, next) => {
   req.templateParams = {
     organisation: orgInfo,
     dataset,
+    issueCount: issues.length ?? 0,
     stats: {
       numberOfFieldsSupplied: numberOfFieldsSupplied ?? 0,
       numberOfFieldsMatched: numberOfFieldsMatched ?? 0,

diff --git a/src/routes/schemas.js b/src/routes/schemas.js
@@ -73,6 +73,7 @@ export const OrgGetStarted = v.strictObject({
 export const OrgDatasetOverview = v.strictObject({
   organisation: OrgField,
   dataset: DatasetNameField,
+  issueCount: v.integer(),
   stats: v.strictObject({
     numberOfRecords: v.integer(),
     numberOfFieldsSupplied: v.integer(),