Skip to content

Fix Shopify test app compile break by exposing ImportPayout in Payments API#8697

Draft
Copilot wants to merge 2 commits into
releases/27.2from
copilot/fix-github-actions-job-failure-another-one
Draft

Fix Shopify test app compile break by exposing ImportPayout in Payments API#8697
Copilot wants to merge 2 commits into
releases/27.2from
copilot/fix-github-actions-job-failure-another-one

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

The Build Apps (W1) (Default) / Apps (W1) (Default) job failed while compiling Shopify tests with AL0132 because Shpfy Payments API did not expose ImportPayout to the test app surface.

  • Root cause

    • ImportPayout was declared local in Shpfy Payments API, but Shopify test code calls it directly.
  • Change

    • In ShpfyPaymentsAPI.Codeunit.al, changed procedure scope:
      • local procedure ImportPayout(...)
      • internal procedure ImportPayout(...)
  • Impact

    • Restores API visibility required by Shopify Connector Test (via internalsVisibleTo) without changing payout import logic.
-    local procedure ImportPayout(JPayout: JsonObject)
+    internal procedure ImportPayout(JPayout: JsonObject)

Copilot AI changed the title [WIP] Fix failing GitHub Actions job Build Apps (W1) Fix Shopify test app compile break by exposing ImportPayout in Payments API Jun 19, 2026
Copilot AI requested a review from djukicmilica June 19, 2026 13:35
@github-actions github-actions Bot added needs-approval Workflow runs require maintainer approval to start AL: Apps (W1) Add-on apps for W1 labels Jun 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234'

@github-actions github-actions Bot removed the needs-approval Workflow runs require maintainer approval to start label Jun 19, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants