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

Commercial invoice simplification #776

Merged
merged 7 commits into from
May 16, 2023
Merged

Conversation

nissimsan
Copy link
Collaborator

This inlines and simplifies and tightens up Commercial Invoice:

  • inlining
  • simplify portOfEntry
  • optional consignee
  • remove postalBox
  • itemsShipped minItems=1
  • remove name, descr, related links
  • country code type
  • req id

@@ -396,12 +395,12 @@ example: |-
}
}
},
"consignee": {
"consignee": [{
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this really a "multiple". field?

Copy link
Collaborator

@mprorock mprorock left a comment

Choose a reason for hiding this comment

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

ok with this, but would prefer either a single consignee, or a way of identifying intermediaries

@@ -396,12 +395,12 @@ example: |-
}
}
},
"consignee": {
"consignee": [{
Copy link
Collaborator

Choose a reason for hiding this comment

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

I suppose if we permit intermediates, but will need to signify ultimately who is final

Copy link
Collaborator

@mkhraisha mkhraisha left a comment

Choose a reason for hiding this comment

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

I think single consignee maps cleaner but we don't currently have a requirement for it.

required:
- type
minItems: 0
maxItems: 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Here

@mkhraisha mkhraisha merged commit b9afc39 into main May 16, 2023
1 check passed
@nissimsan nissimsan deleted the commercial-invoice-simplification branch August 8, 2023 18:42
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.

None yet

5 participants