-
Notifications
You must be signed in to change notification settings - Fork 41
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
[Feature] Break out Net Income from Retained Earnings #57
Comments
Hi @mel-restori I think it may be best if we instead renaming the "Net Income / Retained Earnings Adjustment" field. This field is generated by capturing the Net Income adjustment from the relevant period. This can sometimes be referred to as the retained earnings adjustment as well. However, if you have your own Retained Earnings account (which is seems you do) then I could see the naming of this field to be duplicative. Can you confirm that your end models product a Retained Earnings record for a given period as well as a Net Income / Retained Earnings Adjustment record? If so, then I think this should be an easy feature to roll into our next update to rename the field to |
Hi @fivetran-joemarkiewicz thanks for following up. |
Thanks for this insight @mel-restori! Based on this discovery I think my initial idea would probably be the best course of action to make the record more clear. These "Net Income / Retained Earnings Adjustment" records are simply calculating the "net income" for the periods and factors it into the current BS period. Therefore, renaming the field to just be "Net Income Adjustment" would be more appropriate in my eyes and also allow for it to not be confused with a true Retained Earnings account. |
@fivetran-joemarkiewicz that sounds right, but we also need to get the net income from previous periods in order to match the balance sheet in the Quickbooks UI |
Hi @mel-restori, I'm working on this feature now. Could you elaborate on what you mean about the balance sheet not matching net income from previous periods? It seemed in the previous comment that this solution would work, or do you think there is additional logic required to get the matching you're looking for? |
Hi @fivetran-avinash - I think maybe I don't understand the solution proposed. |
Thanks for the context @mel-restori! We have fixed the field to correctly reflect "Net Income Adjustment". We plan on merging this into our We will try to address creating the Retained Earnings definition you're looking for in a future sprint. Let me know if you have any questions! Happy to assist. |
thanks! |
Hi @mel-restori ! We have deployed this fix in the We are going to close this issue, but if have any questions, thoughts or feedback, please let us know in this issue! Cheers. |
Is there an existing feature request for this?
Describe the Feature
Today, Fivetran combines Net Income and Retained earnings into one line item (called "Net Income / Retained Earnings Adjustment"), but the QuickBooks UI breaks them out separately. It would be helpful for comparisons for them to be broken out. Relevant code is here.
Thank you!
Attaching redacted screenshot from QuickBooks

Describe alternatives you've considered
No response
Are you interested in contributing this feature?
Anything else?
No response
The text was updated successfully, but these errors were encountered: