Skip to content

Truncate Shopify PO Number to External Document No. length - #10305

Open
Sean Schade (seanschade) wants to merge 1 commit into
microsoft:mainfrom
seanschade:fix/shopify-po-number-truncation
Open

Truncate Shopify PO Number to External Document No. length#10305
Sean Schade (seanschade) wants to merge 1 commit into
microsoft:mainfrom
seanschade:fix/shopify-po-number-truncation

Conversation

@seanschade

Copy link
Copy Markdown

Truncate Shopify PO Number to External Document No. length in ShpfyProcessOrder

The PO number coming from Shopify can exceed 35 characters, while Sales Header "External Document No." is Code[35]. The direct Validate call made order creation fail with a string-length error for any B2B order with a long PO number. Truncate with CopyStr/MaxStrLen like the other string mappings in this procedure.

Fixes #10243

What & why

Linked work

Fixes #

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

…ocessOrder

The PO number coming from Shopify can exceed 35 characters, while Sales Header "External Document No." is Code[35]. The direct Validate call made order creation fail with a string-length error for any B2B order with a long PO number. Truncate with CopyStr/MaxStrLen like the other string mappings in this procedure.

Fixes microsoft#10243
@seanschade
Sean Schade (seanschade) requested a review from a team August 17, 2026 13:04
@github-actions github-actions Bot added AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork labels Aug 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Issue #10243 is not valid. Please make sure you link an issue that exists, is open and is approved.

@github-actions github-actions Bot added Integration GitHub request for Integration area needs-approval Workflow runs require maintainer approval to start labels Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork Integration GitHub request for Integration area needs-approval Workflow runs require maintainer approval to start

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Shopify Connector: order sync fails when Shopify PO Number exceeds 35 characters (missing truncation in ShpfyProcessOrder)

1 participant