Skip to content

Commit

Permalink
comment cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
cameroncaci committed Jan 16, 2025
1 parent 26ea095 commit 8d9d3da
Showing 1 changed file with 1 addition and 22 deletions.
23 changes: 1 addition & 22 deletions pkg/services/ppmshipment/payment_packet_creator.go
Original file line number Diff line number Diff line change
Expand Up @@ -141,28 +141,7 @@ func (p *paymentPacketCreator) Generate(appCtx appcontext.AppContext, ppmShipmen

// It was discovered during implementation of B-21938 that watermarks were not functional.
// This is because the watermark func was using bookmarks, not watermarks.
// The func has been removed until this functionality is needed again.
//
// watermarks, err := buildWaterMarks(bookmarks, p.pdfGenerator)
// if err != nil {
// errMsgPrefix = fmt.Sprintf("%s: %s", errMsgPrefix, "failed to generate watermarks for PDF")
// appCtx.Logger().Error(errMsgPrefix, zap.Error(err))
// return nil, fmt.Errorf("%s: %w", errMsgPrefix, err)
// }

// Apply bookmarks and watermarks based on flag
// if addWatermarks && len(watermarks) > 0 {
// pdfWithWatermarks, err := p.pdfGenerator.AddWatermarks(finalMergePdf, watermarks)
// if err != nil {
// errMsgPrefix = fmt.Sprintf("%s: %s", errMsgPrefix, "failed to add watermarks to PDF")
// appCtx.Logger().Error(errMsgPrefix, zap.Error(err))
// return nil, fmt.Errorf("%s: %w", errMsgPrefix, err)
// }
// if addBookmarks {
// return p.pdfGenerator.AddPdfBookmarks(pdfWithWatermarks, bookmarks)
// }
// return pdfWithWatermarks, nil
// }
// See https://github.com/transcom/mymove/pull/14496 for removal

if addBookmarks {
return p.pdfGenerator.AddPdfBookmarks(finalMergePdf, bookmarks)
Expand Down

0 comments on commit 8d9d3da

Please sign in to comment.