-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add created_at
and updated_at
fields
#146
Changes from 5 commits
a89c9e7
3b1d3f4
a736775
9ab70d3
c65bad5
9835893
2e600db
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -1,8 +1,12 @@ | ||||||
# dbt_quickbooks v0.17.0-a1 | ||||||
|
||||||
## Bug Fix | ||||||
- Updated the logic in `int_quickbooks__retained_earnings` to ensure accounting periods with no revenue and expense class lines were accounted for. | ||||||
- This will ensure the net income adjustment is available regardless of existing revenue or expenses. | ||||||
# dbt_quickbooks v0.17.0 | ||||||
[PR #146](https://github.com/fivetran/dbt_quickbooks/pull/146) introduces the following updates: | ||||||
|
||||||
## Breaking Changes | ||||||
- We have introduced the following fields in the `quickbooks__general_ledger` to better analyze real-time transaction data: | ||||||
- `created_at`: The time a transaction was first created. | ||||||
- `updated_at`: The time a transaction was last updated. | ||||||
- This required updates to all our double entry transaction models to bring in these specific fields for each transaction type. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated. |
||||||
- This is a breaking change as this adds new fields to the existing schema for one of our end models. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Changed. |
||||||
|
||||||
# dbt_quickbooks v0.16.0 | ||||||
[PR #143](https://github.com/fivetran/dbt_quickbooks/pull/143) introduces the following updates: | ||||||
|
Large diffs are not rendered by default.
Large diffs are not rendered by default.
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.
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.
Committed!