Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quickstart transformation miscategorizes bill line #105

Open
2 of 4 tasks
rishi-tripathy opened this issue Oct 17, 2023 · 16 comments
Open
2 of 4 tasks

Quickstart transformation miscategorizes bill line #105

rishi-tripathy opened this issue Oct 17, 2023 · 16 comments
Labels
type:wontfix This will not be worked on

Comments

@rishi-tripathy
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Describe the issue

I am working with a customer to bring in Quickbooks data by GL.

I see a transaction the GENERAL_LEDGER transformation as being associated with Account 7410, but in Quickbooks it is clearly classified as 7620.

I believe this is because the inference of the account via the item number associated with the bill line is not happening or is happening incorrectly. The Bill itself is associated with account 7410, but the Bill Line in question is a specific item that should be categorized as 7620.

Relevant error log or model output

As described above, there is a diff where more transactions that expected are being categorized in account 7410, since bill lines that are directly associated with an account are not being categorized as such

Expected behavior

Use the ACCOUNT_EXPENSE_ACCOUNT_ID column on Bill Line table to associate bill line items with accounts as part of the transformations, instead of only relying on the parent bill. line items with direct associations to accounts should override their parent bill.

dbt Project configurations

I'm using Fivetran's quickstart transformations for Quickbooks

Package versions

I'm using Fivetran's quickstart transformations for Quickbooks

What database are you using dbt with?

snowflake

dbt Version

I'm using Fivetran's quickstart transformations for Quickbooks

Additional Context

No response

Are you willing to open a PR to help address this issue?

  • Yes.
  • Yes, but I will need assistance and will schedule time during our office hours for guidance
  • No.
@rishi-tripathy rishi-tripathy added the type:bug Something is broken or incorrect label Oct 17, 2023
@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy thank you for opening the issue--you bring up a very interesting case. I will be looking into this in the next week and will let you know if I have any questions!

@fivetran-catfritz fivetran-catfritz added the status:scoping Currently being scoped label Oct 17, 2023
@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy, following up to my last message, are you seeing this issue in the credit or is this issue with the debit transactions?

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy have you had a chance to check into this? Thanks!

@rishi-tripathy
Copy link
Author

Ah sorry for the delay here Cat! Notifications were piping to my personal email instead of my work one. I'm seeing the impact on debit items in this case. What's the implication of handing them separately?

@fivetran-catfritz
Copy link
Contributor

fivetran-catfritz commented Nov 3, 2023

Thanks @rishi-tripathy ! This is useful to narrow down where the issue might be. I had actually suspected credits, so it's very interesting the issue lies with debits. For debit transactions, we already use account_expense_account_id as the first option to determine account_id for debits (see this line in our model), so I will have to investigate a bit more what could be going on.

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy. I discussed this issue with my team, and it would be really helpful if you could show us live what is happening. Would you be available for a call in the next week? If so, please send an email to solutions@fivetran.com with your availability, and I'll set up the meet!

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy just following up if you would be available for a call with our team?

@fivetran-catfritz fivetran-catfritz added status:stale Issue was blocked or had no user response for more than 30 days and removed status:scoping Currently being scoped labels Nov 20, 2023
@rishi-tripathy
Copy link
Author

rishi-tripathy commented Nov 21, 2023 via email

@fivetran-catfritz
Copy link
Contributor

Thanks @rishi-tripathy! I have scheduled a meeting.

@rishi-tripathy
Copy link
Author

rishi-tripathy commented Nov 27, 2023 via email

@rishi-tripathy
Copy link
Author

rishi-tripathy commented Nov 27, 2023

In this second case (Would need to go back to confirm for the other case) — I can confirm that:

  • the bill line in quickbooks.bill_line is associated with the same account (using account_expense_account_id) that it shows up in the model — though this is different than how it's categorized in the customer account. Asked the customer for more context on why this might be the case.

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy thanks for meeting with us on the call yesterday to go over this issue! To summarize what we found:

  • The 7410 instead of 7620 account was associated with the bill line in question since in the source bill_line table, the account_expense_account_id did not have a value populated.
  • This causes the coalesce clause in this line to move on to another field to determine the account to use.

Next steps:

  • During the call we were unable to establish where the association of 7620 was coming from, so @rishi-tripathy will determine where that path is coming from.

Keep us posted with what you find!

@rishi-tripathy
Copy link
Author

Hey team - have an update here. The discrepancy was being caused because the item_expense_account_id of the item associated with the transactions had changed over time.

That is to say:
When the item was first created (in Feb), it was associated with account 7620
The transaction occured in Feb and Quickbooks correctly associated the transaction with 7620]
at some point between then and now, the item_expense_account_id of the item was updated to be 7410
notably, this doesn't change how the transaction is categorized in Quickbooks.

Now (in November), when the transform runs and sees the item associated with the transaction, it sees a reference to 7410 and therefore puts the transaction in 7410 as per the line referenced

This causes a discrepancy between what hte customer sees in QBO (since the transaction was classified when the item was still associated with 7620), even though the item, today, maps to 7410.

The customer was able to resolve this by overwriting the item associated with the transaction to one with 7620.

I believe that the same customer is experiencing the same issue in a couple of places in their QBO connector, but awaiting confirmation from them. Thanks for investigating together!

@rishi-tripathy
Copy link
Author

Not sure how this could prevented in the future - is historical item data captured somehow?

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy thank you for digging in and summarizing this very interesting find. I spoke with my team, and unfortunately there is currently no historical capture with the connector. Since this is at the connector level, could you open a feature request for the connector itself? You can submit a feature request here, and then I will flag it to the product manager. Thanks again for finding that!

@fivetran-catfritz
Copy link
Contributor

Hi @rishi-tripathy, given the above message, I will mark this ticket as type:wontdo for the time being. If there are developments at the connector level then we will be sure to update the package to reflect the changes.

If anyone else is encountering this issue, please feel free to open a Feature Request for the Fivetran connector so it may be scoped and prioritized accordingly. Thanks!

@fivetran-catfritz fivetran-catfritz added type:wontfix This will not be worked on and removed status:stale Issue was blocked or had no user response for more than 30 days type:bug Something is broken or incorrect labels Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants