Skip to content

bug/dupe-orders#158

Open
fivetran-catfritz wants to merge 7 commits into
mainfrom
bug/dupe-orders
Open

bug/dupe-orders#158
fivetran-catfritz wants to merge 7 commits into
mainfrom
bug/dupe-orders

Conversation

@fivetran-catfritz

@fivetran-catfritz fivetran-catfritz commented May 28, 2026

Copy link
Copy Markdown
Contributor

PR Overview

Package version introduced in this PR:

  • v1.8.2-a1
  • v1.8.2-a2

This PR addresses the following Issue/Feature(s):

  • GA-1026773

Summary of changes:

  • Fixes duplicate customer visits in int_shopify_gql__order by filtering to include only the most recent visit per order, preventing duplicate visits in downstream models.

Submission Checklist

  • SHOPIFY-SPECIFIC: If you added a field to any model with metafields, did you increase the relevant column count in the downstream_model_column_count macro?
  • Alignment meeting with the reviewer (if needed)
    • Timeline and validation requirements discussed
  • Provide validation details:
    • Validation Steps: Check for unintentional effects (e.g., add/run consistency & integrity tests)
    • Testing Instructions: Confirm the change addresses the issue(s)
    • Focus Areas: Complex logic or queries that need extra attention
  • Merge any relevant open PRs into this PR

Changelog

  • Draft changelog for PR
  • Final changelog for release review

@fivetran-catfritz fivetran-catfritz self-assigned this May 28, 2026
@fivetran-catfritz fivetran-catfritz added docs:ready Triggers the docs generator workflow. pre-release Triggers the auto-releaser workflow. labels Jun 1, 2026
Comment thread CHANGELOG.md Outdated
[PR #158](https://github.com/fivetran/dbt_shopify/pull/158) includes the following update:

## Bug Fix
- Fixes duplicate `order_line_id` values in `shopify_gql__order_lines` caused by order lines that belong to multiple fulfillment orders. `int_shopify_gql__order_line` now aggregates `stg_shopify_gql__fulfillment_order_line_item` to the order line grain before joining, so `fulfillable_quantity` correctly reflects the total remaining quantity across all fulfillment orders rather than producing duplicate rows.

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.

Probably less important for a pre-release, but in the interest of making changelogs more concise/less technical, I'm wondering if all the join details are necessary. Might just cut all but the first sentence.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Agreed I will update.

Comment thread README.md Outdated
Comment thread CHANGELOG.md Outdated
Co-authored-by: Savage Fivetran <sarah.savage@fivetran.com>
@fivetran-savage

Copy link
Copy Markdown
Contributor

Approved for pre-release!

@fivetran-catfritz fivetran-catfritz added pre-release Triggers the auto-releaser workflow. and removed pre-release Triggers the auto-releaser workflow. labels Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs:ready Triggers the docs generator workflow. pre-release Triggers the auto-releaser workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants