fivetran / dbt_netsuite

Data models for Fivetran's Netsuite connector, built using dbt.
https://fivetran.github.io/dbt_netsuite/
Apache License 2.0
36 stars 34 forks source link

[Bugfix] Correct translation rate issue on balance sheet #82

Closed fivetran-avinash closed 10 months ago

fivetran-avinash commented 12 months ago

PR Overview

This PR will address the following Issue/Feature: #75

This PR will result in the following new package version: TBD (we are not releasing this version and will be folding it in with #78 once resolved)

LIkely not breaking as this will fix internal logic on one field but not add/remove any current fields.

Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:

PR Checklist

Basic Validation

Please acknowledge that you have successfully performed the following commands locally:

Before marking this PR as "ready for review" the following have been applied:

Detailed Validation

Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

Standard Updates

Please acknowledge that your PR contains the following standard updates:

dbt Docs

Please acknowledge that after the above were all completed the below were applied to your branch:

If you had to summarize this PR in an emoji, which would it be?

👓
fivetran-joemarkiewicz commented 10 months ago

Closing this as PR #92 incorporated these changes