Skip to content

[releases/28.x] [Shopify] Skip FindMapping API calls during customer sync update#7526

Open
onbuyuka wants to merge 1 commit intoreleases/28.xfrom
bugs/629927-backport
Open

[releases/28.x] [Shopify] Skip FindMapping API calls during customer sync update#7526
onbuyuka wants to merge 1 commit intoreleases/28.xfrom
bugs/629927-backport

Conversation

@onbuyuka
Copy link
Copy Markdown
Contributor

@onbuyuka onbuyuka commented Apr 1, 2026

Summary

Backport of bug #625597 to releases/28.x.

Fixes AB#629927

Original PR(s): #7145

## Summary
- When `CreateCustomers = false` (sync/update path), replaced
`FindMapping()` calls with a local DB lookup on `Shpfy Customer` by
`Shop Id` and `Customer SystemId`
- This avoids sending a GraphQL query to Shopify for every BC customer
when running "Sync Customers" with "Can Update Shopify Customer" enabled
- Only customers that already have a Shopify mapping are processed for
updates

Fixes
[AB#625597](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/625597)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@onbuyuka onbuyuka requested a review from a team as a code owner April 1, 2026 14:26
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Apr 1, 2026
@github-actions github-actions bot added this to the Version 28.1 milestone Apr 1, 2026
@onbuyuka onbuyuka enabled auto-merge (squash) April 1, 2026 14:27
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

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.

1 participant