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

add 270 support for U gender #261

Merged
merged 1 commit into from
Sep 14, 2023
Merged

add 270 support for U gender #261

merged 1 commit into from
Sep 14, 2023

Conversation

JakeDMSnap
Copy link
Contributor

No description provided.

Copy link

@jrohrbaugh jrohrbaugh left a comment

Choose a reason for hiding this comment

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

+1 for this change! We just released some code to add U to our 837 request, and it broke our 270 requests. If this is indeed valid, I'd love to see it merged ASAP

@jrohrbaugh
Copy link

@kputnam could you please take a look at this PR? Much appreciated!

@kputnam
Copy link
Owner

kputnam commented Sep 14, 2023

Since I no longer have the X12 spec, I can't check if this conforms to the spec.

I found stedi's documentation includes X279A1, which defines the 270 and 271 transaction sets, but it doesn't show element-level detail. I'm including this link because they have good reference documentation in general and it might be useful to others.

One of CMS's (Medicare) implementation guides does say U can be used for unknown. It would be unusual for Medicare to allow values which aren't allowed by the standard. Medicare is also often regarded as an authority in terms of what X12 should be accepted by commercial payers, so I'm going to say this is good enough evidence to suggest it's probably allowed by the standard.

If anyone does have the X279A1 guide, please let me know if U is not allowed in the implementation section and I will back this change out -- in that case users would need to create payer-specific transaction set grammars by copying the standard and making modifications.

I'll merge this and push a new version tonight. cc: @jrohrbaugh

@kputnam kputnam merged commit 78972b1 into kputnam:master Sep 14, 2023
9 checks passed
@jrohrbaugh
Copy link

confirmed that change fixed our problem. Thanks for the quick and detailed response! When I was researching I also found that CMS guide you shared, but I wasn't sure how official it was. I'll see if anyone at my company has access to an official spec.

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.

3 participants