Closed ccbrandenburg closed 1 year ago
@ccbrandenburg also FYI my team and I will be attending the Coalesce Conference next week and will be having a release freeze in place until we get back. You can expect this change to go live the week of 10/24.
Thanks again so much for your contribution!!
In the meantime, we are good to merge these changes!
Pull Request Are you a current Fivetran customer? Christopher Brandenburg, Gigs (gigs.com), Head of Data
What change(s) does this PR introduce? The
stripe__invoice_line_items.sql
model was missing the invoice's currency.The currency is available further upstream of the model in
stg_stripe__invoice
and I have added the field plus the description.Did you update the CHANGELOG?
Does this PR introduce a breaking change?
stripe__invoice_line_items.sql
. If someone build models on top of this to bring in currency in a different approach this might cause issues.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)
Is this PR in response to a previously created Bug or Feature Request
How did you test the PR changes?
Select which warehouse(s) were used to test the PR
Provide an emoji that best describes your current mood
:cyclone: **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.