fivetran / dbt_stripe

Data models for Stripe built using dbt.
https://fivetran.github.io/dbt_stripe/
Apache License 2.0
30 stars 31 forks source link

Fix timezone support on postgres. Fixes #35 #36

Closed johnf closed 2 years ago

johnf commented 2 years ago

Pull Request Are you a current Fivetran customer? Yes. John Ferlito, Cofounder, Gladly Pty Ltd

What change(s) does this PR introduce? Fixes issues with time zone conversion for postgres

Did you update the CHANGELOG?

Does this PR introduce a breaking change?

If someone on postgres has a timezone set then the package fails to work anyway.

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 :penguin:

fivetran-joemarkiewicz commented 2 years ago

Thanks so much for opening this PR @johnf!! šŸ™Œ

I will have someone from my team review this PR in the coming sprint (starting Thursday) and will let you know if we have any questions before merging and including in our next release šŸ˜„