adds a distinct to the fivetran_utils.string_agg of customer ids in int_shopify__customer_email_rollup
adds extra line to README about the shopify holistic reporting package having a dependency on this package
adds lower() to some spots we are looking at transaction.kind (i have always seen it as lowercase in the source data but just in case -- i changed this while investigating something and felt it didn't hurt to leave in)
remove unused fields from int_shopify__customer_email_rollup
Did you update the CHANGELOG?
[x] Yes
Does this PR introduce a breaking change?
[ ] Yes (please provide breaking change details below.)
[x] No (please provide an explanation as to how the change is non-breaking below.)
no columns were changed -- the customer_ids field will just not aggregate duplicate IDs
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)
[x] Yes
Is this PR in response to a previously created Bug or Feature Request
[ ] Yes, Issue/Feature [link bug/feature number here]
[x] Local (please provide additional testing details below)
before change:
after change:
Aggregated metrics tie out as well
Select which warehouse(s) were used to test the PR
[x] BigQuery
[ ] Redshift
[ ] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)
Provide an emoji that best describes your current mood
👾
**Feedback**
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your [feedback](https://www.surveymonkey.com/r/DQ7K7WW) on our existing dbt packages or what you'd like to see next.
Are you a current Fivetran customer?
internal
What change(s) does this PR introduce?
distinct
to thefivetran_utils.string_agg
of customer ids inint_shopify__customer_email_rollup
lower()
to some spots we are looking at transaction.kind (i have always seen it as lowercase in the source data but just in case -- i changed this while investigating something and felt it didn't hurt to leave in)int_shopify__customer_email_rollup
Did you update the CHANGELOG?
Does this PR introduce a breaking change?
no columns were changed -- the
customer_ids
field will just not aggregate duplicate IDsDid you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)
Is this PR in response to a previously created Bug or Feature Request
How did you test the PR changes?
before change:
after change:
Aggregated metrics tie out as well
Select which warehouse(s) were used to test the PR
Provide an emoji that best describes your current mood
👾 **Feedback** We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your [feedback](https://www.surveymonkey.com/r/DQ7K7WW) on our existing dbt packages or what you'd like to see next.