Skip to content

[releases/27.x] Shopify Payout sync skips records when multiple stores are connected#7686

Merged
onbuyuka merged 4 commits into
releases/27.xfrom
bugs/630989-backport
Apr 29, 2026
Merged

[releases/27.x] Shopify Payout sync skips records when multiple stores are connected#7686
onbuyuka merged 4 commits into
releases/27.xfrom
bugs/630989-backport

Conversation

@onbuyuka
Copy link
Copy Markdown
Contributor

@onbuyuka onbuyuka commented Apr 14, 2026

Summary

Backport of bug #630653 to releases/27.x.

Fixes AB#630994

Original PR(s): #7633

… setups (#7633)

- Add `Shop Code` field to `Shpfy Payout` (30125) and `Shpfy Dispute`
(30155) tables, mirroring the existing pattern on `Shpfy Payment
Transaction`
- Filter `FindLast()` by Shop Code in `ImportNewPayouts()` and
`ImportNewDisputes()` so each store uses its own sync cursor instead of
the global max ID
- Add shop filtering to `UpdatePendingPayouts()`,
`UpdatePaymentTransactionPayoutIds()`, and `UpdateUnfinishedDisputes()`
to prevent cross-store API calls
- Set `Shop Code` on both insert and update paths in `ImportPayout()`
and `ImportDispute()` so existing records are backfilled on first sync
after deployment

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

🤖 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 14, 2026 13:04
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Apr 14, 2026
@onbuyuka onbuyuka marked this pull request as draft April 15, 2026 10:05
@onbuyuka onbuyuka marked this pull request as ready for review April 28, 2026 08:44
@onbuyuka onbuyuka enabled auto-merge (squash) April 28, 2026 08:44
@onbuyuka onbuyuka closed this Apr 28, 2026
auto-merge was automatically disabled April 28, 2026 09:29

Pull request was closed

@onbuyuka onbuyuka reopened this Apr 28, 2026
@onbuyuka onbuyuka enabled auto-merge (squash) April 28, 2026 09:30
Copy link
Copy Markdown
Contributor

@gggdttt gggdttt left a comment

Choose a reason for hiding this comment

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

Looks good

@onbuyuka onbuyuka merged commit 18ad26a into releases/27.x Apr 29, 2026
23 checks passed
@onbuyuka onbuyuka deleted the bugs/630989-backport branch April 29, 2026 13:05
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.

3 participants