Detail what changes this PR introduces and how this addresses the issue/feature request linked above.
The PR creates hierarchical strings for GL accounts and accounting periods and then presents those new fields in the main models. It also adds the commonly used NetSuite fields below into the models.
transaction_accounting_lines.exchange_rate (exchange rate on accounting line)
transaction_lines.is_eliminate (boolean to indicate automatic elimination)
departments.full_name (full hierarchical department name)
subsidiaries_currencies.symbol (base currency of subsidiary)
transaction_lines.netamount (net amount of transaction line)
How did you validate the changes introduced within this PR?
Lyra maintains a fork of this repository. These changes were tested and validated against production NetSuite reports.
Which warehouse did you use to develop these changes?
Snowflake
Did you update the CHANGELOG?
[ ] Yes
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)
[x] Yes
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](https://www.surveymonkey.com/r/DQ7K7WW) on our existing dbt packages or what you'd like to see next.
**PR Template**
- [Community Pull Request Template](?expand=1&template=pull_request_template.md) (default)
- [Maintainer Pull Request Template](?expand=1&template=maintainer_pull_request_template.md) (to be used by maintainers)
Please provide your name and company Jared Monger, Lyra Health
Link the issue/feature request which this PR is meant to address
https://github.com/fivetran/dbt_netsuite/issues/106
Detail what changes this PR introduces and how this addresses the issue/feature request linked above. The PR creates hierarchical strings for GL accounts and accounting periods and then presents those new fields in the main models. It also adds the commonly used NetSuite fields below into the models.
transaction_accounting_lines.exchange_rate
(exchange rate on accounting line)subsidiaries.full_name
(full hierarchical subsidiary name)transaction_lines.is_eliminate
(boolean to indicate automatic elimination)departments.full_name
(full hierarchical department name)subsidiaries_currencies.symbol
(base currency of subsidiary)transaction_lines.netamount
(net amount of transaction line)How did you validate the changes introduced within this PR? Lyra maintains a fork of this repository. These changes were tested and validated against production NetSuite reports.
Which warehouse did you use to develop these changes? Snowflake
Did you update the CHANGELOG?
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)
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](https://www.surveymonkey.com/r/DQ7K7WW) on our existing dbt packages or what you'd like to see next. **PR Template** - [Community Pull Request Template](?expand=1&template=pull_request_template.md) (default) - [Maintainer Pull Request Template](?expand=1&template=maintainer_pull_request_template.md) (to be used by maintainers)