Skip to content

DPC-5246: remove confirmation file generation in opt-out import#2955

Merged
jdettmannnava merged 10 commits intomainfrom
jd/dpc-5246-turn-off-confirmation-file-on-opt-out-import
Apr 3, 2026
Merged

DPC-5246: remove confirmation file generation in opt-out import#2955
jdettmannnava merged 10 commits intomainfrom
jd/dpc-5246-turn-off-confirmation-file-on-opt-out-import

Conversation

@jdettmannnava
Copy link
Copy Markdown
Contributor

@jdettmannnava jdettmannnava commented Mar 20, 2026

🎫 Ticket

https://jira.cms.gov/browse/DPC-5246

🛠 Changes

  • Confirmation file generation and upload removed from opt-out-import
  • utils.go removed from opt-out-import (nothing used in other logic)

ℹ️ Context

When we migrate to the idr-based opt-out-import flow, we won't want to send confirmation files.

🧪 Validation

Integration test shows no confirmation file created. (View first attempt)

@jdettmannnava jdettmannnava changed the title remove confirmation file generation DPC-5246: remove confirmation file generation in opt-out import Mar 20, 2026
@jdettmannnava jdettmannnava marked this pull request as ready for review March 20, 2026 17:51
@jdettmannnava jdettmannnava requested a review from a team as a code owner March 20, 2026 17:51
jscott-nava
jscott-nava previously approved these changes Mar 20, 2026
@jdettmannnava jdettmannnava requested a review from a team March 23, 2026 13:07
ashley-weaver
ashley-weaver previously approved these changes Mar 23, 2026
Copy link
Copy Markdown
Contributor

@ashley-weaver ashley-weaver left a comment

Choose a reason for hiding this comment

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

LGTM

lukey-luke
lukey-luke previously approved these changes Mar 23, 2026
Copy link
Copy Markdown
Contributor

@lukey-luke lukey-luke left a comment

Choose a reason for hiding this comment

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

I verified these changes should be compatible with changes to splunk alerts (DPC-5247).

LGTM!

Copy link
Copy Markdown
Contributor

@ashley-weaver ashley-weaver left a comment

Choose a reason for hiding this comment

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

👍

@jdettmannnava jdettmannnava merged commit 17b8cfc into main Apr 3, 2026
9 checks passed
@jdettmannnava jdettmannnava deleted the jd/dpc-5246-turn-off-confirmation-file-on-opt-out-import branch April 3, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants