Skip to content

changelog: Internal, Reporting,fix IRS Credential Tenure Metric query…#12394

Merged
shilenpatel1 merged 3 commits intomainfrom
155-tenure-metric
Aug 5, 2025
Merged

changelog: Internal, Reporting,fix IRS Credential Tenure Metric query…#12394
shilenpatel1 merged 3 commits intomainfrom
155-tenure-metric

Conversation

@shilenpatel1
Copy link
Contributor

… anb combine report with IRS Fraud Metric

🎫 Ticket

GL-155

Link to the relevant ticket:
GL-155

🛠 Summary of changes

Write a brief description of what you changed.
--> Optimize the Credential Tenure Metric Report Query to run without any timeout issue
--> Update the IRS Fraud Metric Report in order to combin IRS Credential Tenure Metric with IRS Fraud Metric Report
--> Update the Unit Test for IRS Fraud Metric Report in Spec 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)
  • Step 2 Copy the Update IRS Credential Tenure Metric Report and IRS Fraus Metric Report into Rails Console PROD and then Run the report:
    ​​Reports::IrsFraudMetricsReport.new(Date.today).perform_now

👀 Screenshots

Screenshot 2025-08-04 at 3 59 10 PM Screenshot 2025-08-04 at 4 13 55 PM

If relevant, include a screenshot or screen capture of the changes.

Before:
After:

@shilenpatel1 shilenpatel1 force-pushed the 155-tenure-metric branch 4 times, most recently from 5623f7f to 461e74e Compare August 5, 2025 18:13
@shilenpatel1 shilenpatel1 merged commit b6848f9 into main Aug 5, 2025
1 check passed
@shilenpatel1 shilenpatel1 deleted the 155-tenure-metric branch August 5, 2025 20:32
mdiarra3 added a commit that referenced this pull request Aug 7, 2025
* Update translations for the LG33 error page (#12388)

changelog: Upcoming Features, One Account, Update translations for the LG33 error page

* Update json gem (#12386)

changelog: Internal, Maintenance, Update json gem

* Upgrade saml_idp to 0.23.9-18f (#12389)

changelog: Internal, Security, Update ruby-saml gem

* Undo unintended resolution proofing change (#12390)

* Undo unintended changes from One Account work

changelog: Bug Fixes, Resolution Proofing, Undo unintended changes from One Account work

These changes from commit f628580 were
not intended.

* Reinstate dependency so that cascaded deletes can happen

* Lg-16484 create new alert banner for dos api down (#12387)

* changelog: Internal, Doc Auth, adding logging for step in dos api health check

* Adding step as analytics argument

* Fixing specs

* Updated param comment and spec

* Adding back arguments in choose id type concern spec

* Adding specs for logging

* Resolving PR comments

* Fix LG-99 metric in Fraud Report | GL-Data-184 (#12396)

Fixes uppercase typo 
Changes query order for efficiency 

changelog: Internal, Reporting, Fix LG-99 metric in Fraud Report

Refer to: https://gitlab.login.gov/lg-teams/Team-Data/reporting/-/issues/184
Co-authored-by: shilen.patel <shilen.patel@gsa.gov>

* changelog: Internal, Doc Auth, adding spec to confirm mobile flow routes from how to verify to doc capture after ipp canceled (#12393)

* changelog: Internal, Reporting,fix IRS Credential Tenure Metric query… (#12394)

* changelog: Internal, Reporting,fix IRS Credential Tenure Metric query anb combine report with IRS Fraud Metric

* changelog: Internal, Reporting,fix spec file

* changelog: Internal, Reporting,fix lint issue

* Check for duplicates only within requesting service provider (#12391)

changelog: Upcoming Features, One Account, Check for duplicates only within requesting service provider

Fixes https://cm-jira.usa.gov/browse/LG-16327

---------

Co-authored-by: Vraj Mohan <vraj.mohan@gsa.gov>
Co-authored-by: Mitchell Henke <mitchell.henke@gsa.gov>
Co-authored-by: A Shukla <abir.shukla@gsa.gov>
Co-authored-by: Gerardo E. Cruz-Ortiz <59618057+astrogeco@users.noreply.github.com>
Co-authored-by: shilen.patel <shilen.patel@gsa.gov>
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