[Feature] Update `intercompany` categorization

[fivetran-catfritz]

Is there an existing feature request for this?

• [X] I have searched the existing issues

Describe the Feature

Currently, netsuite2__transaction_details considers an account as intercompany if the account name contains the word “intercompany”. Reviewing the netsuite documentation (see here and here) and per internal discussion, we should be using the eliminate column from the source account table.

We will need to:

1. Bring in eliminate column to stg_netsuite2__accounts (also should check on netsuite if this applies)
2. Use eliminate to determine is_account_intercompany flag. The source uses the letters T and F (well our source does, so maybe consider if it can also come in as boolean), so convert to boolean in transform.

Describe alternatives you've considered

No response

Are you interested in contributing this feature?

• [X] Yes.
• [ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
• [ ] No.

Anything else?

No response

[jmongerlyra]

I agree with this approach.

[jmongerlyra]

What's the timeline on this fix? It is actually needed in our environment for all three models.

[fivetran-joemarkiewicz]

@jmongerlyra we will plan to include this update in the next major release of netsuite. We are waiting to validate the results with our internal finance team before rolling them out.

I expect by the end of our current sprint (10/18) we are hoping to have all updates rolled out to the next version of the Netsuite package.

[fivetran-joemarkiewicz]

This feature has been incorporated in the latest release of the dbt_netsuite and dbt_netsuite_source packages. These updates should be live at the top of the hour.