Skip to content

changelog: Internal, Reporting, Update IRS Verification Report#12249

Merged
shilenpatel1 merged 2 commits intomainfrom
113_update_irs_verification
Jun 11, 2025
Merged

changelog: Internal, Reporting, Update IRS Verification Report#12249
shilenpatel1 merged 2 commits intomainfrom
113_update_irs_verification

Conversation

@shilenpatel1
Copy link
Copy Markdown
Contributor

@shilenpatel1 shilenpatel1 commented Jun 9, 2025

🎫 Ticket

GL-113
Link to the relevant ticket:
LG-XXXXX

🛠 Summary of changes

Write a brief description of what you changed.
--> Update funnel_table metric names irs_verfication_report
--> Updated the unit tests for irs_verfication_report inside irs_verfication_report_spec.rb file

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Step 1 - Run unit tests for spec files in this PR (i.e. bundle exec rspec ./spec/path/to/spec_file)

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.
Screenshot 2025-06-09 at 5 57 32 PM
Screenshot 2025-06-09 at 6 57 20 PM

Before:
After:

@shilenpatel1 shilenpatel1 force-pushed the 113_update_irs_verification branch from 5cfcddf to 9a117da Compare June 10, 2025 19:23
Copy link
Copy Markdown
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

@shilenpatel1 shilenpatel1 merged commit 5c57436 into main Jun 11, 2025
1 check passed
@shilenpatel1 shilenpatel1 deleted the 113_update_irs_verification branch June 11, 2025 18:08
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