FEATURE - Add Passthrough Columns to Locations Model

[oreoana]

Are you a Fivetran customer?
Yes -- Blue Bottle Coffee

Is your feature request related to a problem? Please describe.
I'd like to be able to add pass-through columns to some of the Netsuite staging models, specifically stg_netsuite__locations.

Describe the solution you'd like
Be able to specify pass-through columns for the locations staging model in the netsuite_source package using the existing approach here: https://github.com/fivetran/dbt_netsuite_source#configuration

Describe alternatives you've considered
Build our own Netsuite locations staging model.

Please indicate the level of urgency and business impact of this request
Medium -- would allow us to set up a better infrastructure for our locations data pipeline.

Are you interested in contributing to this package?

• Yes, I can do this and open a PR for your review.
• Possibly, but I'm not quite sure how to do this. I'd be happy to do a live coding session with someone to get this work implemented.
• No, I'd prefer if someone else did this. I don't have the time and/or don't know how to incorporate the changes necessary.

[fivetran-joemarkiewicz]

Hi @oreoana

Thanks so much for opening this feature request! I have added this feature request to our backlog and we will let you know back here once we are able to assign this request to a future sprint.

However, in the meantime as this is a relatively easy addition to the package (and I see you are open to contributing) then I would highly recommend opening a PR against the master branch for this addition. You can follow the same process of this variable as shown in stg_netsuite__accounts model and name the variable in line with the other ones. I am thinking locations_pass_through_columns. Let me know if you have any questions, otherwise we will let you know here once we are able to tackle this request in a future sprint.

Thank you!

[fivetran-joemarkiewicz]

@oreoana I actually am working on a similar PR #10 where the pass through columns for customers is being added. I will build off this PR and add the locations variable as well.

[fivetran-joemarkiewicz]

@oreoana I wanted to provide an FYI that the locations_pass_through_columns variable was included in the latest release of the dbt_netsuuite_source package. You can expect the package to be live on the dbt hub at the top of the hour.

Thanks again for opening this issue! 😄

[oreoana]

@fivetran-joemarkiewicz great! Thanks so much for this!