Skip to content

changelog: Internal, Reporting, Update IRS Authentication Report#12258

Merged
shilenpatel1 merged 1 commit intomainfrom
112-Update-irs-auth-report
Jun 16, 2025
Merged

changelog: Internal, Reporting, Update IRS Authentication Report#12258
shilenpatel1 merged 1 commit intomainfrom
112-Update-irs-auth-report

Conversation

@shilenpatel1
Copy link
Copy Markdown
Contributor

🎫 Ticket

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

🛠 Summary of changes

Write a brief description of what you changed.
--> Update names of the metric in irs_authentication_report
> Change "System and Application Demand" to read "Registration demand"
> Change "System and application errors" to read "Registration failures"
> Change "Authentication attempts" to read "Registration successes"
> Change "Authentication success rate" to read "Registration success rate"
--> Updated the unit tests for irs_authentication_report inside irs_authentication_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-12 at 1 44 56 PM
Screenshot 2025-06-12 at 1 47 05 PM

Before:
After:

@shilenpatel1 shilenpatel1 requested a review from edimeo-gsa June 12, 2025 20:28
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.

Everything looks good to me

@shilenpatel1 shilenpatel1 merged commit acee57d into main Jun 16, 2025
1 check passed
@shilenpatel1 shilenpatel1 deleted the 112-Update-irs-auth-report branch June 16, 2025 14:57
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