Skip to content

update dmf pvc post call reports for v3 evidences#6086

Merged
jacobkagon merged 4 commits intotrunkfrom
CU_868hvnqf5_update_dmf_pvc_post_call_reports
Mar 16, 2026
Merged

update dmf pvc post call reports for v3 evidences#6086
jacobkagon merged 4 commits intotrunkfrom
CU_868hvnqf5_update_dmf_pvc_post_call_reports

Conversation

@TristanB17
Copy link
Copy Markdown
Contributor

@TristanB17 TristanB17 commented Mar 13, 2026

PR Checklist

Please check if your PR fulfills the following requirements:

  • The title follows our guidelines
  • Tests for the changes have been added (for bug fixes/features), and they use let helpers and before blocks.
  • For all UI changes, there is Cucumber coverage.
  • Any endpoint touched in the PR has an appropriate Pundit policy. For open endpoints, the reasoning is documented in the PR and code.
  • Any endpoint modified in the PR only responds to the expected MIME types.
  • For all scripts or rake tasks, how to run them is documented in both the PR and the code.
  • There are no inline styles added.
  • There is no inline JavaScript added.
  • There is no hard-coded text added/updated in helpers/views/JavaScript. New/updated translation strings do not include markup/styles unless there is supporting documentation.
  • Code does not use .html_safe.
  • All images added/updated have alt text.
  • Does not bypass RuboCop rules in any way.

PR Type

What kind of change does this PR introduce?:

  • Bugfix
  • Feature (requires Feature flag)
  • Data fix, Migration, or Report (inert code, no impact until run)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • CI related changes
  • Dependency updates (e.g., add a new gem or update to a version)
  • Release (Prepares code for a release, e.g., version bumps, changelog updates, tagging, deployment scripts)

What is the ticket # detailing the issue?

Ticket: https://app.clickup.com/t/868hvnqf5

A brief description of the changes:

Current behavior: reports are not accurately compiled due to referencing the pre-v3 evidence data structure

New behavior: reports have been updated to use the v3 evidence data structure

Feature Flag

For all new feature development, a feature flag is required to control the exposure of the feature to our end users. A feature flag needs a corresponding environment variable to initialize the state of the flag. Please share the name of the environment variable below that would enable/disable the feature and indicate which client(s) it applies to.

Variable name:

  • DC
  • ME

Additional Context

Include any additional context that may be relevant to the peer review process.

saipraveen18
saipraveen18 approved these changes Mar 16, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@saipraveen18 saipraveen18 left a comment

Choose a reason for hiding this comment

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

lgtm 👍

ATBull81
ATBull81 approved these changes Mar 16, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@ATBull81 ATBull81 left a comment

Choose a reason for hiding this comment

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

adds default for report, grabs alive_status from app and not person, lgtm barring specs!

@jacobkagon jacobkagon merged commit 68397ab into trunk Mar 16, 2026
392 of 407 checks passed
@jacobkagon jacobkagon deleted the CU_868hvnqf5_update_dmf_pvc_post_call_reports branch March 16, 2026 16:34
sri49 pushed a commit that referenced this pull request Mar 23, 2026
@TristanB17 @sri49
update dmf pvc post call reports for v3 evidences (#6086)
ade7f86 
* update dmf post response report to use v3 evidences

* update pvc and dmf evidence reports for V3 evidences

* readd removed spec file
sri49 pushed a commit that referenced this pull request Mar 30, 2026
@TristanB17 @sri49
update dmf pvc post call reports for v3 evidences (#6086)
9afeff2 
* update dmf post response report to use v3 evidences

* update pvc and dmf evidence reports for V3 evidences

* readd removed spec file
vFos2016 pushed a commit that referenced this pull request Apr 8, 2026
@sri49 @TristanB17
@saipraveen18 @ATBull81 @jacobkagon @mdkaraman @polographer @charlienparker @satish2025-crew @vkghub @kristinmerbach
CU-868j243qf :: 31.7.5 pre release (#6140)
ec7773d 
* set new rop due date for bulk calls (#6102)

* save

* updates evidence spec

* remove utils changes, add specs

* update specs

* remove binding

* fix error

* add specs for non esi bulk calls

* remove local adjustments to run spec

---------

Co-authored-by: Sai Praveen Gudimetla <saipraveen.gudimetla@gmail.com>

* address gap in pvc logic (#6100)

* save

* updates evidence spec

* separate rrv other bulk call updates

---------

Co-authored-by: Sai Praveen Gudimetla <saipraveen.gudimetla@gmail.com>

* remove custom handling on remove qhp applicant modal (#6108)

Co-authored-by: Alec Turnbull <alecturnbull81@gmail.com>

* refactor access verification logic in FamiliesController (#6115)

* refactor access verification logic in FamiliesController

* fix spec

* remediate CVE-2026-33306 with bcrypt bump to 3.1.22 (#6120)

CVE-2026-33306 bcrypt minor version uplift

* data fix ridp verified consumers with bad bookmark url (#6121)

* data fix for bad bookmark urls

* update scope of fix to all ridp verified with bad bookmark

* rubocop fix

* set admin bookmark url to nil

* security: fix bundler-audit vulnerabilities (activestorage) (#6133)

fix: update gems to resolve bundler-audit security vulnerabilities

This commit updates vulnerable gems identified by bundler-audit.
See PR description for detailed analysis of changes and affected components.

* script to remove duplicate mailing addresses (#6028)

* remove duplicate mailing addresses

* usage fix

* spec fix

* fix missing qhp applications in eligibility report (#6031)

* add spec for non year match case

* add year filter based on application

* spec fix

* remove active_applicant call in favor of applicant, refactor for clarity

* pass in applicant to aca method, spec coverage

* update dmf pvc post call reports for v3 evidences (#6086)

* update dmf post response report to use v3 evidences

* update pvc and dmf evidence reports for V3 evidences

* readd removed spec file

* add rake task to generate loacl mec evidence report (#6059)

* add rake task to generate loacl mec evidence report

* move logic to dedicated class

* rubocop fix

---------

Co-authored-by: vishal kalletla <vishuk1201@gmail.com>

* add verification history prior to sending dmf request (#6137)

* add verification history prior to sending dmf request

* update class comparison check

* fetch latest determined qhp application in cv payload (#6009)

* fetch latest determined qhp application in cv payload

* lint fix

* manual verification required feature (#6111)

* manual verification required feature

* system file

* forgot to push spec files

* reintroduce disabled FAA continue button (#6132)

* redirect ridp verified consumers on pre ridp pages (#6124)

* redirect ridp verified consumers

* spec fix

* cucumber coverage

* update spec to accurately expect outstanding aasm state (#6156)

* update test to accurately expected evidence aasm state

* fix indentation on spec

* remove unnecessary line from spec

* add script to manually extend individual market evidences (#6022)

* add script to manually extend individual market evidences

* lint fix

* filter out non enrolled consumers in extend evidences script (#6073)

* filter out non enrolled consumers in extend evidences script

* fix whitespace

* fix whitespace

* Add script to delete stale tax household enrollments (#6157)

* Add script to delete stale tax household enrollments

* fix comment

---------

Co-authored-by: Tristan <33517218+TristanB17@users.noreply.github.com>
Co-authored-by: Sai Praveen Gudimetla <saipraveen.gudimetla@gmail.com>
Co-authored-by: Alec Turnbull <alecturnbull81@gmail.com>
Co-authored-by: Jacob Kagon <69021620+jacobkagon@users.noreply.github.com>
Co-authored-by: Michael Karamanov <mdkaramanov@gmail.com>
Co-authored-by: Marco (Polo) Ornelas <marco.ornelas@ideacrew.com>
Co-authored-by: Charlie Parker <charlie.parker@ideacrew.com>
Co-authored-by: satish2025-crew <satish.polamuri@ideacrew.com>
Co-authored-by: vishal kalletla <vishuk1201@gmail.com>
Co-authored-by: kristinmerbach <78749008+kristinmerbach@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@saipraveen18 saipraveen18 saipraveen18 approved these changes

@ATBull81 ATBull81 ATBull81 approved these changes

Assignees

No one assigned

Labels

bugfix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@TristanB17 @saipraveen18 @ATBull81 @jacobkagon