Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PACA-465: enabling SAMHSA filtering on all Claim/ClaimResponse calls #766

Merged
merged 1 commit into from
Mar 2, 2022

Conversation

ericbuckley
Copy link
Contributor

@ericbuckley ericbuckley commented Feb 24, 2022

Fixes PACA-465

We need to exclude all claims that have SAMHSA diagnosis and/or procedure codes in it. The filtering was added to the BFD API in the Fall of 2021, but never enabled here.

Proposed Changes

On all Claim or ClaimResponse API calls, we should be sending the query parameter excludeSAMHSA=true to filter out claims with SAMHSA codes.

Change Details

Security Implications

  • new software dependencies
  • security controls or supporting software altered
  • new data stored or transmitted
  • security checklist is completed for this change
  • requires more information or team discussion to evaluate security implications
  • no PHI/PII is affected by this change

Acceptance Validation

Feedback Requested

@ericbuckley ericbuckley added bug Something isn't working paca Feature to support the PACA Project labels Feb 24, 2022
@ericbuckley ericbuckley self-assigned this Feb 24, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #766 (cbdac24) into master (cc65f01) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
+ Coverage   66.26%   66.27%   +0.01%     
==========================================
  Files          65       65              
  Lines        5825     5827       +2     
==========================================
+ Hits         3860     3862       +2     
  Misses       1505     1505              
  Partials      460      460              
Impacted Files Coverage Δ
bcda/client/bluebutton.go 85.71% <100.00%> (+0.13%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc65f01...cbdac24. Read the comment docs.

@ericbuckley ericbuckley marked this pull request as ready for review February 24, 2022 20:33
@ericbuckley ericbuckley changed the title PACA-465: (DRAFT) enabling SAMHSA filtering on all Claim/ClaimResponse calls PACA-465: enabling SAMHSA filtering on all Claim/ClaimResponse calls Feb 24, 2022
Copy link
Contributor

@whytheplatypus whytheplatypus left a comment

Choose a reason for hiding this comment

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

lgtm

@ericbuckley ericbuckley merged commit 50f009a into master Mar 2, 2022
@ericbuckley ericbuckley deleted the ericbuckley/paca-465-enable-samhsa branch March 2, 2022 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working paca Feature to support the PACA Project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants