feat: add passthrough columns to card model.

[bramrodenburg]

Please provide your name and company

Bram - Gigs

Link the issue/feature request which this PR is meant to address

#80

Detail what changes this PR introduces and how this addresses the issue/feature request linked above.

Add passthrough variables for the stg_stripe__card model. This can, for example, be used to pull in non-standard columns from Stripe such as description, iin and issuer.

How did you validate the changes introduced within this PR?

Followed the instructions here and tested the changes against our own internal data. Note, I could only do this for iin and issuer, not for description.

Which warehouse did you use to develop these changes?

BigQuery

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

Typically there are additional maintenance changes required before this will be ready for an upcoming release. Are you comfortable with the Fivetran team making a few commits directly to your branch?

• Yes
• No

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

🐸

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.

PR Template

• Community Pull Request Template (default)

• Maintainer Pull Request Template (to be used by maintainers)

[fivetran-jamie]

Thanks @bramrodenburg for opening this! Looks great.

One question - Regarding description, did you just not include it in your tests, or was it throwing an error or something like that?

Followed the instructions here and tested the changes against our own internal data. Note, I could only do this for iin and issuer, not for description.

Actually, second question -- are there downstream models in the stripe transform package you'd want these passthrough columns to persist to? Such as stripe__balance_transactions. If so, I can include it in my other stripe PR

[fivetran-jamie]

Looks and runs great, just a couple of documentation-related suggestions, and see above questions ^

[bramrodenburg]

One question - Regarding description, did you just not include it in your tests, or was it throwing an error or something like that?

I didn't include it in our tests. We asked Stripe at the time to enable the iin and issuer columns, but not the description column. So that's why I didn't include that column in the test.

Actually, second question -- are there downstream models in the stripe transform package you'd want these passthrough columns to persist to? Such as stripe__balance_transactions. If so, I can include it in my other stripe PR

At the moment not. Thanks for asking :)

Looks and runs great, just a couple of documentation-related suggestions, and see above questions ^

Committed your documentation-related suggestions!

[fivetran-jamie]

Woo thank you! This looks great to me. I'm going to merge it into a release branch to make some other changes and CI tests.

Many thanks