Skip to content
This repository has been archived by the owner on Mar 1, 2021. It is now read-only.

account for 2012-07 #2

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

account for 2012-07 #2

wants to merge 9 commits into from

Conversation

kaguillera
Copy link
Contributor

The initial entries for transactions for the 2012-07 month. @whit537 Please give me the Gittip DB entries for that payday. Thanks

2012/07.dat Outdated

; IHasAMoney.com
2012-07-02 Users
Assets:Operations:Stripe $ 2.92 ; Assuming a stripe deposit.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This'll be Samurai, actually. IHAM never moved to Stripe.

@chadwhitacre chadwhitacre changed the title 2012-07 Transactions reconcile 2012-07 Jan 13, 2016
@chadwhitacre
Copy link
Contributor

Here's a snip from the reconciliation dashboard I'm working on:

Date         Charges  Payouts  Income   Escrow Δ
------------------------------------------------
2012-07-06    375.24     0.00   33.63     341.61
2012-07-11      0.00    24.30    0.00     -24.30
2012-07-13    555.27     0.00   46.14     509.13
2012-07-20    509.51     0.00   46.29     463.22
2012-07-27  1,623.21     0.00   99.31   1,523.90

@kaguillera
Copy link
Contributor Author

Date         Charges  Payouts  Income   Escrow Δ
------------------------------------------------
2012-07-06    375.24     0.00   33.63     341.61

For the above data taken from the gittip database we should have charged users cards $375.24 for which $341.61 would go to the persons they wish to tip and the $33.63 used to pay Stripe for processing the credit cards of the users. However the actual fees that Stripe took based on the sum of fees for that data transactions from [https://github.com/gratipay/logs/blob/master/statements/csv/2012/07/stripe.csv] it is only $30.11 and we actually get $345.13 in our New Alliance account. So the question is how do we account for that?
P.S. See no Evernote 😃

@kaguillera
Copy link
Contributor Author

So the $345.13 which is the total charged by Stripe to users minus the fees they take match the amount deposited in New Alliance account ( this is expected ).
What happened on the Gittip DB?

@kaguillera
Copy link
Contributor Author

The question is how much should the data in the Gittip database affect or be reflected on the Accounting system. Since the actual money is in the Stripe and New Alliance accounts it just means that our internal records (Gittip database ) may be wrong. Do we need to make sure that it is 100% correct?

@kaguillera
Copy link
Contributor Author

At the risk of adding insult to injury the second-to-last record in the table i.e.

Date         Charges  Payouts  Income   Escrow Δ
------------------------------------------------
2012-07-20    509.51     0.00   46.29     463.22

Does not match the Stripe records. New Alliance shows a deposit of $468.06 and Stripe records show that the deposit should have been $468.64. Only a difference of $0.58, but a difference non the less. Thus all three sources in information have different values.

@kaguillera
Copy link
Contributor Author

Am I missing something??

Date         Charges  Payouts  Income   Escrow Δ
------------------------------------------------
2012-07-27  1,623.21     0.00   99.31   1,523.90

The above is from Gittip data right??
Strip data shows that the amount charged to users was $1117.26. Did Paypal make up the difference?

@chadwhitacre
Copy link
Contributor

P.S. See no Evernote 😃

!m @kaguillera 💃

@chadwhitacre
Copy link
Contributor

Do we need to make sure that it is 100% correct?

Yes! :)

@chadwhitacre
Copy link
Contributor

What happened on the Gittip DB?

Don't forget the processing fee buffer!

@chadwhitacre
Copy link
Contributor

Ftr we ended up cleaning up #1 and not even really getting to the above issues.

@chadwhitacre
Copy link
Contributor

And I guess that's because, in not forgetting the processing fee buffer, we realized that the fee buffer needed some cleanup.

@chadwhitacre chadwhitacre mentioned this pull request Jan 18, 2016
65 tasks
@chadwhitacre
Copy link
Contributor

Rebased on master. Old HEAD was 04ac73f.

@chadwhitacre chadwhitacre force-pushed the 2012-07 branch 2 times, most recently from 522eb2b to 5040a61 Compare January 27, 2016 20:40
@chadwhitacre
Copy link
Contributor

Rebased on master again, to pick up #4. Old HEAD was 522eb2b.

@chadwhitacre
Copy link
Contributor

Wait a sec! Why does this pass?!

$ gb
* 2012-07
  2012-08
  master
$ test.py 
test_net_income_reconciles_with_retained_earnings ... -4658.76 -4658.76
test_income_balances ... good
test_expenses_balance ... good
test_fee_buffer_reconciles ... 402.32 -812.30 -409.98 -71.98 338.00
Traceback (most recent call last):
  File "./bin/test.py", line 108, in <module>
    test_func()
  File "./bin/test.py", line 74, in test_fee_buffer_reconciles
    assert delta == fee_buffer
AssertionError

test_escrow_balances ... 4705.08 -5043.08
Traceback (most recent call last):
  File "./bin/test.py", line 108, in <module>
    test_func()
  File "./bin/test.py", line 43, in test_escrow_balances
    assert escrow_assets + escrow_liability == 0
AssertionError

$

@chadwhitacre
Copy link
Contributor

That's better. :)

@kaguillera
Copy link
Contributor Author

Figured out the reason for the discrepancy (#2 (comment))

Date         Charges  Payouts  Income   Escrow Δ
------------------------------------------------
2012-07-27  1,623.21     0.00   99.31   1,523.90

It turns out that from that payday till (I don't know when) wee started using two payment venders (Stripe and Balanced). But there is another problem now. The records from New Alliance only shows the deposit from Stripe of ~$1030 but there is no record of the rest of funds being deposited in New Alliance from Balanced. @whit537 is there another account that Balanced would have deposited to?

@chadwhitacre
Copy link
Contributor

Figured out the reason for the discrepancy

!m @kaguillera

is there another account that Balanced would have deposited to?

I believe we didn't start withdrawing funds from Balanced until a while later, since Balanced was designed to hold funds for us (withdrawing funds was a manual process).

@kaguillera
Copy link
Contributor Author

ok cool so we have to add this account to the .dat files.

Another note! We need to remember that from payday 8 we also started charging users a minimum of $10.00 even though there tips were much lower. I am just putting this here as a reminder that we have to deal with that when entering transactions in the .dat files. I think that you would agree that remainder of the funds would reside in the Liabilities:Escrow:New Alliance/Balanced

@chadwhitacre
Copy link
Contributor

Agreed.

@chadwhitacre chadwhitacre changed the title reconcile 2012-07 account for 2012-07 Nov 30, 2016
@kaguillera
Copy link
Contributor Author

Just making a note that we have ported to beancount so this PR will be changing

@kaguillera
Copy link
Contributor Author

Now this PR is blocked by (to be created). The old new reconciliation dashboard does not show
amount payins by payday, vendor and amount which is needed to correctly account for the finances Escrow etc. So the new PR (to be created) should do this.

Blocked by gratipay.#4342
@kaguillera
Copy link
Contributor Author

So just checked Stripe records on line and the 2012-07- 06 hit on 2012-07-20
and also 2012-07-13 also hit on 2012-07-20.

@kaguillera
Copy link
Contributor Author

kaguillera commented Feb 23, 2017

Ready for review @whit537, @nobodxbodon pending merging of gratipay/gratipay.com#4342 for the verification gratipay payments amount.

Copy link
Contributor

@nobodxbodon nobodxbodon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

About closing balance of new Alliance: in statement it's $1,372.49, but assets of balance sheet is 1456.44-83.49=1372.95

; IHasAMoney
; ==========

2012-07-02 * "We charge a user vis Samurai."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: vis -> with?

; Gittip Payday 5
; ===============

2012-07-06 * "We Charged participants via Stripe"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: charged

Assets:Escrow:Stripe 341.61 USD
Assets:Operations:Stripe 33.63 USD
Income:Fees:Stripe -33.63 USD
Liabilities:Escrow -341.61 USD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: align please

; Gittip Payday 6
; ===============

2012-07-13 * "We Charged participants via Stripe"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: charged

Income:Fees:Stripe -33.63 USD
Liabilities:Escrow -341.61 USD

2012-07-19 * "Stripe settlement the net to us"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: settled

Assets:Escrow:Stripe 1027.29 USD
Assets:Operations:Stripe 59.97 USD
Income:Fees:Stripe -59.97 USD
Liabilities:Escrow -1027.29 USD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: align please

Liabilities:Escrow -1027.29 USD


; Startubg to use Balanced as Payment provider
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Started?

; ============================================

; Accounts to open next month
; 2012-07-27 open Expenses:Fees:Balanced USD
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this be moved to 2012-08.beancount in the other PR?

Copy link
Contributor Author

@kaguillera kaguillera Apr 7, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes...just made a reminder for me. But I will remove now since I have already started working on the 2012-08.beancount PR.

@kaguillera
Copy link
Contributor Author

Sorry for all the little errors was on a bus at the time. Will fix these errors

@kaguillera kaguillera removed the Review label Mar 2, 2017
@kaguillera
Copy link
Contributor Author

Blocked for now until gratipay/gratipay.com#3912 is resolved

@kaguillera
Copy link
Contributor Author

About closing balance of new Alliance: in statement it's $1,372.49, but assets of balance sheet is 1456.44-83.49=1372.95

This was corrected in ad77314. Please check it was the appropriate changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants