Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update shopify__daily_shop model to include new 'delayed' status from Shopify #81

Conversation

shreveasaurus
Copy link
Contributor

Description
Update test to include new 'delayed' status from Shopify.

Brent Shreve
Onebridge - https://www.onebridge.tech/

Link the issue/feature request which this PR is meant to address
fivetran/dbt_shopify_source#83

Detail what changes this PR introduces and how this addresses the issue/feature request linked above.
This PR updates the shopify__daily_shop model to include the new 'delayed' fulfillment status from Shopify.

How did you validate the changes introduced within this PR?
In a development and production environment where Fivetran and dbt are deployed for Shopify.

Which warehouse did you use to develop these changes?
Snowflake

Did you update the CHANGELOG?

  • Yes

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

  • Yes

Provide an emoji that best describes your current mood

🥞

@shreveasaurus shreveasaurus marked this pull request as ready for review May 30, 2024 14:52
@fivetran-jamie fivetran-jamie changed the base branch from main to releases/v0.12.latest June 7, 2024 18:59
@fivetran-jamie fivetran-jamie self-requested a review June 7, 2024 19:17
Copy link
Contributor

@fivetran-jamie fivetran-jamie left a comment

Choose a reason for hiding this comment

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

looks great!! gonna merge into my release branch to combine with some other updates and apply some light polishes

@fivetran-jamie fivetran-jamie merged commit 7a4acbd into fivetran:releases/v0.12.latest Jun 7, 2024
@fivetran-jamie fivetran-jamie mentioned this pull request Jun 7, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants