Skip to content

Report fix#12250

Merged
koseni123 merged 2 commits intomainfrom
irs_verfication_report_fix2
Jun 10, 2025
Merged

Report fix#12250
koseni123 merged 2 commits intomainfrom
irs_verfication_report_fix2

Conversation

@koseni123
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket:
LG-47

🛠 Summary of changes

Changed location of definition table
Updated export file to use csv

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Step 1: Ensure spec file passes
  • Step 2: Ensure report is able to run queries and generates email
  • Step 3: ensure email sends per schedule in the config (Weekly)

👀 Screenshots

main

Copy link
Contributor

@edimeo-gsa edimeo-gsa left a comment

Choose a reason for hiding this comment

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

LGTM

@koseni123 koseni123 merged commit 8b93428 into main Jun 10, 2025
1 check passed
@koseni123 koseni123 deleted the irs_verfication_report_fix2 branch June 10, 2025 14:36
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.

2 participants