Skip to content

Commit 4d4b76e

Browse files
Merge pull request #14 from fivetran/feature/gl-batch-addition
Feature/gl batch addition
2 parents 78b6109 + 2543c8b commit 4d4b76e

23 files changed

+299
-60
lines changed

.github/PULL_REQUEST_TEMPLATE/maintainer_pull_request_template.md

Lines changed: 11 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -4,48 +4,27 @@
44
**This PR will result in the following new package version:**
55
<!--- Please add details around your decision for breaking vs non-breaking version upgrade. If this is a breaking change, were backwards-compatible options explored? -->
66

7-
**Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:**
7+
**Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:**
8+
<!--- Copy/paste the CHANGELOG for this version below. -->
89

910
## PR Checklist
1011
### Basic Validation
1112
Please acknowledge that you have successfully performed the following commands locally:
12-
- [ ] dbt compile
13-
- [ ] dbt run –full-refresh
14-
- [ ] dbt run
15-
- [ ] dbt test
16-
- [ ] dbt run –vars (if applicable)
13+
- [ ] dbt run –full-refresh && dbt test
14+
- [ ] dbt run (if incremental models are present)
1715

1816
Before marking this PR as "ready for review" the following have been applied:
19-
- [ ] The appropriate issue has been linked and tagged
20-
- [ ] You are assigned to the corresponding issue and this PR
21-
- [ ] BuildKite integration tests are passing
17+
- [ ] The appropriate issue has been linked, tagged, and properly assigned.
18+
- [ ] All necessary documentation and version upgrades have been applied.
19+
<!--- Be sre to update the package version in the dbt_project.yml, integration_tests/dbt_project.yml, and README if necessary. -->
20+
- [ ] docs were regenerated (unless this PR does not include any code or yml updates).
21+
- [ ] BuildKite integration tests are passing.
22+
- [ ] Detailed validation steps have been provided below.
2223

2324
### Detailed Validation
24-
Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":
25-
- [ ] You have validated these changes and assure this PR will address the respective Issue/Feature.
26-
- [ ] You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
27-
- [ ] You have provided details below around the validation steps performed to gain confidence in these changes.
25+
Please share any and all of your validation steps:
2826
<!--- Provide the steps you took to validate your changes below. -->
2927

30-
### Standard Updates
31-
Please acknowledge that your PR contains the following standard updates:
32-
- Package versioning has been appropriately indexed in the following locations:
33-
- [ ] indexed within dbt_project.yml
34-
- [ ] indexed within integration_tests/dbt_project.yml
35-
- [ ] CHANGELOG has individual entries for each respective change in this PR
36-
<!--- If there is a parallel upstream change, remember to reference the corresponding CHANGELOG as an individual entry. -->
37-
- [ ] README updates have been applied (if applicable)
38-
<!--- Remember to check the following README locations for common updates. →
39-
<!--- Suggested install range (needed for breaking changes) →
40-
<!--- Dependency matrix is appropriately updated (if applicable) →
41-
<!--- New variable documentation (if applicable) -->
42-
- [ ] DECISIONLOG updates have been updated (if applicable)
43-
- [ ] Appropriate yml documentation has been added (if applicable)
44-
45-
### dbt Docs
46-
Please acknowledge that after the above were all completed the below were applied to your branch:
47-
- [ ] docs were regenerated (unless this PR does not include any code or yml updates)
48-
4928
### If you had to summarize this PR in an emoji, which would it be?
5029
<!--- For a complete list of markdown compatible emojis check our this git repo (https://gist.github.com/rxaviers/7360908) -->
5130
:dancer:

.github/workflows/auto-release.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
name: 'auto release'
2+
on:
3+
pull_request:
4+
types:
5+
- closed
6+
branches:
7+
- main
8+
9+
jobs:
10+
call-workflow-passing-data:
11+
if: github.event.pull_request.merged
12+
uses: fivetran/dbt_package_automations/.github/workflows/auto-release.yml@feature/auto-releaser
13+
secrets: inherit

.gitignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
target/
22
dbt_packages/
33
logs/
4-
.DS_Store
4+
.DS_Store
5+
env/
6+
package-lock.yml

CHANGELOG.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,24 @@
1+
# dbt_sage_intacct_source v0.3.0
2+
3+
[PR #14](https://github.com/fivetran/dbt_sage_intacct_source/pull/14) includes the following updates.
4+
5+
## 🚨 Breaking Changes 🚨:
6+
- Removal of the `_fivetran_deleted` field from the `stg_sage_intacct__gl_detail` table due to this field being deprecated within the connector. The relevant information is now available within the `gl_batch` source table.
7+
8+
## Bug Fixes
9+
- Cast the `class_id` field within the following models as `{{ dbt.type_string() }}` in order to ensure datatype consistency for the field across models. This also ensures downstream transformation logic within `dbt_sage_intacct` compiles successfully.
10+
- `stg_sage_intacct__ap_bill_item`
11+
- `stg_sage_intacct__ar_invoice_item`
12+
13+
## Feature Updates
14+
- Addition of the `gl_batch` source and relevant downstream models:
15+
- `stg_sage_intacct__gl_batch_tmp`
16+
- `stg_sage_intacct__gl_batch`
17+
18+
## Under the Hood
19+
- Updated Maintainer PR Template
20+
- Included auto-releaser GitHub Actions workflow to automate future releases
21+
122
# dbt_sage_intacct_source v0.2.1
223

324
A very spoooOOOOOooooky release 🎃 👻

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ If you are **not** using the [Sage Intacct transformation package](https://githu
3939
```yaml
4040
packages:
4141
- package: fivetran/sage_intacct_source
42-
version: [">=0.2.0", "<0.3.0"] # we recommend using ranges to capture non-breaking changes automatically
42+
version: [">=0.3.0", "<0.4.0"] # we recommend using ranges to capture non-breaking changes automatically
4343
```
4444
## Step 3: Define database and schema variables
4545
By default, this package runs using your destination and the `sage_intacct` schema. If this is not where your Sage Intacct data is (for example, if your Sage Intacct schema is named `sage_intacct_fivetran`), add the following configuration to your root `dbt_project.yml` file:

dbt_project.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: 'sage_intacct_source'
2-
version: '0.2.1'
2+
version: '0.3.0'
33
config-version: 2
44
require-dbt-version: [">=1.3.0", "<2.0.0"]
55
models:
@@ -11,6 +11,7 @@ models:
1111
vars:
1212
sage_intacct_source:
1313
gl_detail: "{{ source('sage_intacct','gl_detail') }}"
14+
gl_batch: "{{ source('sage_intacct','gl_batch') }}"
1415
gl_account: "{{ source('sage_intacct','gl_account') }}"
1516
ap_bill_item: "{{ source('sage_intacct','ap_bill_item') }}"
1617
ap_bill: "{{ source('sage_intacct','ap_bill') }}"

docs/catalog.json

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

docs/index.html

Lines changed: 2 additions & 2 deletions
Large diffs are not rendered by default.

docs/manifest.json

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)