aggregate distinct customer ids in email rollup

[fivetran-jamie]

Are you a current Fivetran customer?

internal

What change(s) does this PR introduce?

• 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?

• Yes

Does this PR introduce a breaking change?

• Yes (please provide breaking change details below.)
• 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)

• Yes

Is this PR in response to a previously created Bug or Feature Request

• Yes, Issue/Feature [link bug/feature number here]
• No but came out of validating the solution for [Bug] Duplicate Rows in shopify_holistic_reporting__daily_customer_metrics dbt_shopify_holistic_reporting#14 and [Feature] update package for v0.8.0 of shopify package dbt_shopify_holistic_reporting#12

How did you test the PR changes?

• Buildkite
• 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

• 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 on our existing dbt packages or what you'd like to see next.

[fivetran-avinash]

Hey @fivetran-jamie! So it looks like the customer ids are distinct now, but the aggregation is still not changing and doesn't match with the order counts and amount summations for customer ids with multiple tags. Can you take a look at the issues I saw in int_shopify__customer_email_rollup and see if there's validity to what I'm seeing? Thanks!

[fivetran-avinash]

All's good! Whenever Databricks stops timing out you're good to release.