-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding subscription_id in invoice table for use in the main stripe pa… #41
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ashishk-de thanks so much for opening this PR 🙌
I apologize for the delayed response in giving this a review. I just took a look and this enhancement seems like a more than reasonable addition to the package. Before we progress with merging this to main
and including in the next release, I did have a few requested updates:
- With the new field being added to the staging model, we should also update the
src_stripe.yml
andstg_stripe.yml
files to include the new field within the dbt documentation. - Make the same version increase adjustment that you made in the
dbt_project.yml
file within theintegration_test/dbt_project.yml
file.
Once those small changes are applied, we should be good to go! Let me know if you have any questions 😄
This has been merged with #45 |
closing as this was merged with another PR |
…ckage
Pull Request
Are you a current Fivetran customer?
yes. Abine Inc
What change(s) does this PR introduce?
Adds subscription_id to invoice table.
Did you update the CHANGELOG?
Does this PR introduce a breaking change?
My stripe instance uses subscription. I am unsure if the Invoice object from stripe will always have the subscription_id column available based on your usage of subscriptions feature in stripe.
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
💃
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.