Feature: Upload series (#327)

api/src/graphql/uploads.sdl.ts

@@ -16,6 +16,7 @@ export const schema = gql`
     signedUrl: String
     latestValidation: UploadValidation
     subrecipientUploads: [SubrecipientUpload!]
+    seriesUploads: [Upload]
   }
 
   type Query {

api/src/services/uploads/uploads.test.ts

@@ -1,5 +1,3 @@
-import { Upload } from '@prisma/client'
-
 import {
   deleteUploadFile,
   s3UploadFilePutSignedUrl,
@@ -104,7 +102,7 @@ describe('uploads', () => {
     'returns all uploads in their agency for organization staff',
     async (scenario: StandardScenario) => {
       mockCurrentUser(scenario.user.three)
-      const result: Upload[] = await uploads()
+      const result = await uploads()
 
       const uploadsBelongToOrg = await uploadsBelongToOrganization(
         result,
@@ -151,7 +149,7 @@ describe('uploads', () => {
   })
 
   scenario('updates an upload', async (scenario: StandardScenario) => {
-    const original = (await upload({ id: scenario.upload.one.id })) as Upload
+    const original = await upload({ id: scenario.upload.one.id })
     const result = await updateUpload({
       id: original.id,
       input: { filename: 'String2' },
@@ -168,55 +166,79 @@ describe('uploads', () => {
     expect(deleteUploadFile).not.toHaveBeenCalled()
   })
 
-  scenario(
-    'returns the latest validation for an upload when there are multiple validations',
-    async (scenario: StandardScenario) => {
-      const uploadIdOne = scenario.upload.one.id
-      const latestValidationOfScenarioOne =
-        await UploadRelationResolver.latestValidation(
-          {},
-          {
-            root: {
-              id: uploadIdOne,
-              agencyId: scenario.upload.one.agencyId,
-              createdAt: scenario.upload.one.createdAt,
-              filename: scenario.upload.one.filename,
-              reportingPeriodId: scenario.upload.one.reportingPeriodId,
-              updatedAt: scenario.upload.one.updatedAt,
-              uploadedById: scenario.upload.one.uploadedById,
-            },
-            context: undefined,
-            info: undefined,
-          }
+  describe('Uploads resolvers', () => {
+    scenario(
+      'latest validations: returns the latest validation for an upload when there are multiple validations',
+      async (scenario: StandardScenario) => {
+        const uploadIdOne = scenario.upload.one.id
+        const latestValidationOfScenarioOne =
+          await UploadRelationResolver.latestValidation(
+            {},
+            {
+              root: {
+                id: uploadIdOne,
+                agencyId: scenario.upload.one.agencyId,
+                createdAt: scenario.upload.one.createdAt,
+                filename: scenario.upload.one.filename,
+                reportingPeriodId: scenario.upload.one.reportingPeriodId,
+                updatedAt: scenario.upload.one.updatedAt,
+                uploadedById: scenario.upload.one.uploadedById,
+              },
+              context: undefined,
+              info: undefined,
+            }
+          )
+
+        const uploadIdTwo = scenario.upload.two.id
+        const latestValidationOfScenarioTwo =
+          await UploadRelationResolver.latestValidation(
+            {},
+            {
+              root: {
+                id: uploadIdTwo,
+                agencyId: scenario.upload.two.agencyId,
+                createdAt: scenario.upload.two.createdAt,
+                filename: scenario.upload.two.filename,
+                reportingPeriodId: scenario.upload.two.reportingPeriodId,
+                updatedAt: scenario.upload.two.updatedAt,
+                uploadedById: scenario.upload.two.uploadedById,
+              },
+              context: undefined,
+              info: undefined,
+            }
+          )
+
+        expect(latestValidationOfScenarioOne?.createdAt).toEqual(
+          new Date('2024-01-27T10:32:00.000Z')
+        )
+        expect(latestValidationOfScenarioTwo?.createdAt).toEqual(
+          new Date('2024-01-29T18:13:25.000Z')
         )
+      }
+    )
 
-      const uploadIdTwo = scenario.upload.two.id
-      const latestValidationOfScenarioTwo =
-        await UploadRelationResolver.latestValidation(
+    scenario(
+      'uploads resolver: correctly returns series uploads',
+      async (scenario: StandardScenario) => {
+        const results = await UploadRelationResolver.seriesUploads(
           {},
           {
             root: {
-              id: uploadIdTwo,
-              agencyId: scenario.upload.two.agencyId,
-              createdAt: scenario.upload.two.createdAt,
-              filename: scenario.upload.two.filename,
-              reportingPeriodId: scenario.upload.two.reportingPeriodId,
-              updatedAt: scenario.upload.two.updatedAt,
-              uploadedById: scenario.upload.two.uploadedById,
+              ...scenario.upload.one,
             },
             context: undefined,
             info: undefined,
           }
         )
 
-      expect(latestValidationOfScenarioOne?.createdAt).toEqual(
-        new Date('2024-01-27T10:32:00.000Z')
-      )
-      expect(latestValidationOfScenarioTwo?.createdAt).toEqual(
-        new Date('2024-01-29T18:13:25.000Z')
-      )
-    }
-  )
+        expect(results).toHaveLength(2)
+        expect(results.map((seriesUpload) => seriesUpload.id)).toEqual([
+          scenario.upload.two.id,
+          scenario.upload.one.id,
+        ])
+      }
+    )
+  })
 })
 
 describe('downloads', () => {
@@ -334,7 +356,7 @@ describe('getValidUploadsInCurrentPeriod', () => {
 
       const uploads = await getValidUploadsInCurrentPeriod(
         scenario.organization.one,
-        upload.reportingPeriodId
+        scenario.reportingPeriod.one
       )
 
       expect(uploads.map((upload) => upload.id)).toContain(upload.id)
@@ -351,7 +373,7 @@ describe('getValidUploadsInCurrentPeriod', () => {
 
       const uploads = await getValidUploadsInCurrentPeriod(
         scenario.organization.two,
-        upload.reportingPeriodId
+        scenario.reportingPeriod.one
       )
 
       expect(uploads.map((upload) => upload.id)).toContain(upload.id)
@@ -367,7 +389,7 @@ describe('getValidUploadsInCurrentPeriod', () => {
 
       const uploads = await getValidUploadsInCurrentPeriod(
         scenario.organization.two,
-        upload.reportingPeriodId
+        scenario.reportingPeriod.one
       )
 
       expect(uploads.map((upload) => upload.id)).not.toContain(upload.id)

api/src/services/uploads/uploads.ts

@@ -4,6 +4,7 @@ import cloneDeep from 'lodash/cloneDeep'
 import type {
   QueryResolvers,
   MutationResolvers,
+  ResolversTypes,
   UploadRelationResolvers,
 } from 'types/graphql'
 import { v4 as uuidv4 } from 'uuid'
@@ -130,6 +131,7 @@ export const downloadUploadFile: MutationResolvers['downloadUploadFile'] =
     return signedUrl
   }
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 export const Upload: UploadRelationResolvers = {
   uploadedBy: (_obj, { root }) => {
     return db.upload.findUnique({ where: { id: root?.id } }).uploadedBy()
@@ -157,6 +159,23 @@ export const Upload: UploadRelationResolvers = {
     })
     return latestValidation
   },
+  seriesUploads: async (
+    _obj,
+    { root }
+  ): Promise<ResolversTypes['Upload'][]> => {
+    return db.upload.findMany({
+      where: {
+        AND: {
+          agencyId: root?.agencyId,
+          expenditureCategoryId: root?.expenditureCategoryId,
+          reportingPeriodId: root?.reportingPeriodId,
+        },
+      },
+      orderBy: {
+        createdAt: 'desc',
+      },
+    })
+  },
 }
 type UploadsWithValidationsAndExpenditureCategory = Prisma.UploadGetPayload<{
   include: { validations: true; expenditureCategory: true }

web/src/components/ReportingPeriod/ReportingPeriods/ReportingPeriods.tsx

@@ -2,6 +2,7 @@ import { useState, useMemo, useCallback } from 'react'
 
 import Button from 'react-bootstrap/Button'
 import Modal from 'react-bootstrap/Modal'
+import type { FindReportingPeriodsWithCertification } from 'types/graphql'
 import { useAuth } from 'web/src/auth'
 
 import { useMutation } from '@redwoodjs/web'
@@ -136,7 +137,9 @@ const ReportingPeriodsList = ({
           </Button>
         </Modal.Footer>
       </Modal>
-      <TableBuilder
+      <TableBuilder<
+        FindReportingPeriodsWithCertification['reportingPeriodsWithCertification']
+      >
         data={reportingPeriods}
         columns={columns}
         filterableInputs={['name']}

web/src/components/ReportingPeriod/ReportingPeriods/columns.tsx

@@ -1,10 +1,14 @@
 import { createColumnHelper } from '@tanstack/react-table'
+import type { FindReportingPeriodsWithCertification } from 'types/graphql'
 
 import { Link, routes } from '@redwoodjs/router'
 
 import { formatDateString } from 'src/utils'
 
-const columnHelper = createColumnHelper()
+const columnHelper =
+  createColumnHelper<
+    FindReportingPeriodsWithCertification['reportingPeriodsWithCertification']
+  >()
 
 export const columnDefs = ({ certificationDisplay, canEdit, isUSDRAdmin }) => {
   const columns = [

web/src/components/Subrecipient/SubrecipientTableUploadLinksDisplay/SubrecipientTableUploadLinksDisplay.tsx

@@ -1,14 +1,14 @@
-import { Subrecipient } from 'types/graphql'
+import { FindSubrecipients } from 'types/graphql'
 
 import { Link, routes } from '@redwoodjs/router'
 
 type SubrecipientUpload =
-  | Subrecipient['validSubrecipientUploads'][number]
-  | Subrecipient['invalidSubrecipientUploads'][number]
+  | FindSubrecipients['subrecipients'][number]['validSubrecipientUploads'][number]
+  | FindSubrecipients['subrecipients'][number]['invalidSubrecipientUploads'][number]
 
 interface SubrecipientTableUploadLinksDisplayProps {
-  validSubrecipientUploads: Subrecipient['validSubrecipientUploads']
-  invalidSubrecipientUploads: Subrecipient['invalidSubrecipientUploads']
+  validSubrecipientUploads: FindSubrecipients['subrecipients'][number]['validSubrecipientUploads']
+  invalidSubrecipientUploads: FindSubrecipients['subrecipients'][number]['invalidSubrecipientUploads']
 }
 
 const SubrecipientTableUploadLinksDisplay = ({

web/src/components/Subrecipient/Subrecipients/Subrecipients.tsx

@@ -1,3 +1,5 @@
+import type { FindSubrecipients } from 'types/graphql'
+
 import TableBuilder from 'src/components/TableBuilder/TableBuilder'
 
 import { columnDefs } from './columns'
@@ -6,7 +8,7 @@ const Subrecipients = ({ subrecipients }) => {
   const filterableInputs = ['uei', 'tin']
 
   return (
-    <TableBuilder
+    <TableBuilder<FindSubrecipients['subrecipients'][number]>
       data={subrecipients}
       columns={columnDefs}
       filterableInputs={filterableInputs}

web/src/components/Subrecipient/Subrecipients/columns.tsx

@@ -1,10 +1,11 @@
 import { createColumnHelper, ColumnDef } from '@tanstack/react-table'
-import { ParsedSubrecipient, Subrecipient } from 'types/graphql'
+import { ParsedSubrecipient, FindSubrecipients } from 'types/graphql'
 
 import SubrecipientTableUploadLinksDisplay from 'src/components/Subrecipient/SubrecipientTableUploadLinksDisplay/SubrecipientTableUploadLinksDisplay'
 import { formatDateString, formatPhoneNumber } from 'src/utils'
 
-const columnHelper = createColumnHelper<Subrecipient>()
+const columnHelper =
+  createColumnHelper<FindSubrecipients['subrecipients'][number]>()
 
 function formatDetails(
   parsedSubrecipient: ParsedSubrecipient | null | undefined
@@ -46,7 +47,9 @@ const getUEI = (ueiTinCombo: string) => ueiTinCombo.split('_')[0]
 
 const getTIN = (ueiTinCombo: string) => ueiTinCombo.split('_')[1]
 
-export const columnDefs: ColumnDef<Subrecipient>[] = [
+export const columnDefs: ColumnDef<
+  FindSubrecipients['subrecipients'][number]
+>[] = [
   columnHelper.accessor((row) => getUEI(row.ueiTinCombo), {
     id: 'uei',
     cell: (info) => info.getValue() ?? '',

web/src/components/TableBuilder/TableBuilder.tsx

@@ -6,6 +6,7 @@ import {
   getSortedRowModel,
   getFilteredRowModel,
   ColumnFiltersState,
+  ColumnDef,
 } from '@tanstack/react-table'
 import { Button } from 'react-bootstrap'
 import Table from 'react-bootstrap/Table'
@@ -18,11 +19,21 @@ import TableRow from './TableRow'
   and sorting functionality.
   For documentation, visit: https://tanstack.com/table/v8/docs/guide/introduction
 */
-function TableBuilder({ data, columns, filterableInputs = [] }) {
+interface TableBuilderProps<T> {
+  data: T[]
+  columns: ColumnDef<T>[]
+  filterableInputs?: string[]
+}
+
+function TableBuilder<T extends object>({
+  data,
+  columns,
+  filterableInputs = [],
+}: TableBuilderProps<T>) {
   const [columnFilters, setColumnFilters] = useState<ColumnFiltersState>([])
   const [sorting, setSorting] = useState([])
 
-  const table = useReactTable({
+  const table = useReactTable<T>({
     data,
     columns: columns,
     state: {
@@ -41,7 +52,7 @@ function TableBuilder({ data, columns, filterableInputs = [] }) {
   }
 
   return (
-    <div className="p-2">
+    <div className="pt-2">
       <Table striped borderless>
         <thead>
           {!!filterableInputs.length && (

web/src/components/TableBuilder/TableRow.tsx

@@ -3,11 +3,20 @@
 const TableRow = ({ row }) => {
   return (
     <tr key={row.id}>
-      {row.getVisibleCells().map((cell) => (
-        <td key={cell.id} className="border border-slate-700">
-          {flexRender(cell.column.columnDef.cell, cell.getContext())}
-        </td>
-      ))}
+      {row.getVisibleCells().map((cell) => {
+        const cellProps =
+          typeof cell.column.columnDef.meta?.getCellProps === 'function'
+            ? cell.column.columnDef.meta.getCellProps(cell)
+            : {}
+
+        cellProps.className = `${cellProps.className || ''
+          } border border-slate-700`
+        return (
+          <td key={cell.id} {...cellProps}>
+            {flexRender(cell.column.columnDef.cell, cell.getContext())}
+          </td>
+        )
+      })}
     </tr>
   )
 }