-
Notifications
You must be signed in to change notification settings - Fork 32
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
Jm updates #8
Jm updates #8
Conversation
invoice.amount_remaining, | ||
invoice.attempt_count, | ||
invoice.description as invoice_memo, | ||
invoice_line_item.unique_id as invoice_line_item_id, |
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.
This update was to reflect bug noticed by Issue #3
invoice_line_item.period_start, | ||
invoice_line_item.period_end, |
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.
This update was to reflect enhancement noted in Issue #5
Will want to add final model field name changes to release notes |
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.
Minor change suggestions, otherwise looks good!
.circleci/config.yml
Outdated
echo `pwd` | ||
cd integration_tests | ||
dbt deps | ||
dbt seed --target bigquery |
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.
dbt seed --target bigquery | |
dbt seed --target bigquery --full-refresh |
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.
Updated
dbt_project.yml
Outdated
|
||
require-dbt-version: [">=0.18.0", "<0.20.0"] | ||
|
||
models: | ||
stripe: | ||
+schema: _stripe |
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.
+schema: _stripe | |
+schema: stripe |
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.
Updated
This PR includes the following updates: