Skip to content

8477 s3 refactor#9385

Open
AbelTaxCourt wants to merge 116 commits intostagingfrom
8477-s3-refactor
Open

8477 s3 refactor#9385
AbelTaxCourt wants to merge 116 commits intostagingfrom
8477-s3-refactor

Conversation

@AbelTaxCourt
Copy link
Contributor

@AbelTaxCourt AbelTaxCourt commented Nov 17, 2025

  • Added documentStorageId field to the Docket Entry entity instead of using docketEntryId as the S3 key so docket entries on unconsolidated cases can update their own S3 documents independently from the consolidated group
    (Supports Consolidation: Change to QC/Edit Docket Entry Process #8477)
  • This PR includes the second migration that backfills the documentStorageId column on the Docket Entry table using docketEntryId
  • This PR includes the functionality that uses the new documentStorageId field as the key when retrieving S3 documents
  • See 8477 s3 first migration #9706

AbelTaxCourt and others added 9 commits November 12, 2025 14:17
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
…teractor

Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
AbelTaxCourt and others added 19 commits November 17, 2025 17:08
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
…ntext to preserve cerebral-integration test alias logic and prevent jest compile errors
…he correct key upon document upload

Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
…Interactor, remove duplicate trial session file

Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
AbelTaxCourt and others added 4 commits February 24, 2026 13:34
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
if (stampedPdf) {
originalPdfDoc = await PDFDocument.load(stampedPdf);
} else {
const { documentStorageId } = caseEntity.docketEntries.find(de => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could use the new getDocumentStorageId utility here (apologies if this is a duplicate comment, something glitched for me in vs code)

});

if (!docketEntryEntity) {
docketEntryEntity = caseEntity.docketEntries.find(de => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could use the new utility here, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants