Feature/update joins

[fivetran-catfritz]

PR Overview

This PR will address the following Issue/Feature: #12

This PR will result in the following new package version:

• 1.1
• Not breaking since it is a join update, not adding or removing columns.

Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:

• Updated the join logic for recharge__subscription_overview to produce more accurate results.
• Since we now know purchase_item_id = subscription_id for subscriptions. Update join from:

    from subscriptions
    left join customers_charge_lines
        on customers_charge_lines.customer_id = subscriptions.customer_id
        and customers_charge_lines.address_id = subscriptions.address_id
        and customers_charge_lines.external_product_id_ecommerce = subscriptions.external_product_id_ecommerce

to

    from subscriptions
    left join customers_charge_lines
        on customers_charge_lines.purchase_item_id = subscriptions.subscription_id

PR Checklist

Basic Validation

Please acknowledge that you have successfully performed the following commands locally:

• dbt compile
• dbt run –full-refresh (ok but no incrementals)
• dbt run
• dbt test
• dbt run –vars (if applicable)

Before marking this PR as "ready for review" the following have been applied:

• The appropriate issue has been linked and tagged
• You are assigned to the corresponding issue and this PR
• BuildKite integration tests are passing

Detailed Validation

Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

• You have validated these changes and assure this PR will address the respective Issue/Feature.
• You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
• You have provided details below around the validation steps performed to gain confidence in these changes.

• See internal ticket

Standard Updates

Please acknowledge that your PR contains the following standard updates:

• Package versioning has been appropriately indexed in the following locations:
  • indexed within dbt_project.yml
  • indexed within integration_tests/dbt_project.yml
• CHANGELOG has individual entries for each respective change in this PR
• README updates have been applied (if applicable)
• DECISIONLOG updates have been updated (if applicable)
• Appropriate yml documentation has been added (if applicable)

dbt Docs

Please acknowledge that after the above were all completed the below were applied to your branch:

• docs were regenerated (unless this PR does not include any code or yml updates)

If you had to summarize this PR in an emoji, which would it be?

💃

[fivetran-jamie]

lgtm! just a changelog suggestion and 1 question to better understand the change

[fivetran-jamie]

obligatory reminder to update before release