Skip to content

Irs verification report update#12178

Merged
sb-2011 merged 9 commits intomainfrom
irs_verification_report_update
May 19, 2025
Merged

Irs verification report update#12178
sb-2011 merged 9 commits intomainfrom
irs_verification_report_update

Conversation

@koseni123
Copy link
Contributor

@koseni123 koseni123 commented May 14, 2025

🎫 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.

Looks good, approved.

KolawoleHOseni added 3 commits May 15, 2025 17:40
@edimeo-gsa edimeo-gsa self-requested a review May 19, 2025 12:27
@sb-2011 sb-2011 merged commit 9103fcb into main May 19, 2025
1 check passed
@sb-2011 sb-2011 deleted the irs_verification_report_update branch May 19, 2025 19:41
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