Skip to content

[Shopify] Fix payout and dispute sync skipping records in multi-store setups#7633

Open
onbuyuka wants to merge 1 commit intomainfrom
bugs/630653-payout-sync-shop-code
Open

[Shopify] Fix payout and dispute sync skipping records in multi-store setups#7633
onbuyuka wants to merge 1 commit intomainfrom
bugs/630653-payout-sync-shop-code

Conversation

@onbuyuka
Copy link
Copy Markdown
Contributor

@onbuyuka onbuyuka commented Apr 10, 2026

Summary

  • 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

🤖 Generated with Claude Code

@onbuyuka onbuyuka requested a review from a team as a code owner April 10, 2026 09:54
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Apr 10, 2026
… setups

Add Shop Code field to Payout and Dispute tables and filter all sync
procedures by shop, preventing the SinceId cursor from crossing stores.
Existing records are backfilled on first re-import via the update path.

Fixes AB#630653

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@onbuyuka onbuyuka force-pushed the bugs/630653-payout-sync-shop-code branch from 4b0216c to 45f9f65 Compare April 10, 2026 09:59
@github-actions github-actions bot added this to the Version 29.0 milestone Apr 10, 2026
@onbuyuka onbuyuka enabled auto-merge (squash) April 10, 2026 11:15
key(Key1; Date)
{
}
key(Key2; "Shop Code") { }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Formatting

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