Skip to content

NIMTP-2909 Parse PER Administrative Communications Contact - #156

Open
x0l08og wants to merge 5 commits into
walmartlabs:masterfrom
x0l08og:2909
Open

NIMTP-2909 Parse PER Administrative Communications Contact#156
x0l08og wants to merge 5 commits into
walmartlabs:masterfrom
x0l08og:2909

Conversation

@x0l08og

@x0l08og x0l08og commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@CLAassistant

CLAassistant commented Dec 10, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

// PER03/05/07, Code identifying the type of communication number
private String communicationNumberQualifier;

// PER04/06/08, Complete communications number including country or area code when applicable

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It can be telephone number or email, can we change these to communicationDetailQualifier and communicationDetail or communicationInfoQualifier and communicationInfo

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I got this name communicationNumberQualifier/communicationNumber from 856 Ship Notice/Manifest pdf..

N1*SF*1 EDI SOURCE
N3*31875 SOLON ROAD
N4*SOLON*OH*44139
PER*PY*FSMA CONTACT*TE*+18884636332*EA*FoodSafetyPlanBuilder@fda.hhs.gov

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add a test case with PER segment at the item level with UR qualifier?

txSegments.add(new X12Segment("HL*12*11*I"));
txSegments.add(new X12Segment("LIN**UP*039364133147"));
txSegments.add(new X12Segment("SN1**2*EA"));
txSegments.add(new X12Segment("N1*SF*RESER'S FINE FOODS, INC.*UL*0090266420001"));

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

At item level N101 should be MP or ZW

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants