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

account for 2017-01 #38

Closed
wants to merge 12 commits into from
Closed

account for 2017-01 #38

wants to merge 12 commits into from

Conversation

nobodxbodon
Copy link
Contributor

@whit537 @kaguillera could you please review while I continue, so that we can find issue earlier? I put FD # and record in csv (in log repository) as beancount comment for easier cross-check.

@chadwhitacre
Copy link
Contributor

So exciting! 😍 💃

2017-01-01 open Expenses:Operations USD
2017-01-01 open Income:Payout USD

; date based on receipt in FD #6502. Corresponding record in pnc.csv L16
Copy link
Contributor

@chadwhitacre chadwhitacre Feb 9, 2017

Choose a reason for hiding this comment

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

How about just including the actual links?

; https://gratipay.freshdesk.com/helpdesk/tickets/6502
; https://github.com/gratipay/logs/blob/master/statements/2017/01/pnc.csv#L16

Copy link
Contributor

Choose a reason for hiding this comment

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

Even better: can we include these URL references using Beancount metadata?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done in 30ba400. Thanks.

Expenses:Operations 34.15 USD
Assets:Operations:PNC -34.15 USD

; TODO: find receipt for pnc.csv L17
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be under MaxCDN or StackPath, if that helps.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right. Added.

; in order to achieve opening balance on 2017-01-01, add difference between 2883.99 and the latest balance of PNC (currently 0)
2017-01-01 * "Opening Balance"
Assets:Operations:PNC 2883.99 USD
Equity:Owners:Chad-Whitacre -2883.99 USD
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@whit537 does this make sense? This is to cover the change to PNC account before 2017.

@nobodxbodon
Copy link
Contributor Author

There are some paypal activities (pay out, transfer) that are not captured yet. Some will have corresponding record in New Alliance statement.

@nobodxbodon nobodxbodon mentioned this pull request Feb 12, 2017
65 tasks
@nobodxbodon
Copy link
Contributor Author

@whit537 @kaguillera may I ask for some suggestion to handle paypal activity like https://gratipay.freshdesk.com/helpdesk/tickets/6588, and braintree transaction like https://github.com/gratipay/logs/blob/master/statements/2017/01/new-alliance.csv#L3?

@nobodxbodon
Copy link
Contributor Author

@whit537 @kaguillera is it possible/necessary to export Paypal statements so that I can proceed? AFAIK processing the braintree transactions are blocked by #3 (comment), and it can take longer to unblock?

@chadwhitacre
Copy link
Contributor

Sorry I haven't gotten to this yet @nobodxbodon, trying to get back here ...

@nobodxbodon
Copy link
Contributor Author

nobodxbodon commented Feb 22, 2017

@whit537 without continous book-keeping, it seems tricky to get the open liability of 201701. For now all the balances are booked like they are from your pocket. In 436f0e1 I put an estimated liability as one part of paypal's balance. Note: the first payout booked in bb6d073 doesn't include the fee and I'll add that in the next commit.

@nobodxbodon
Copy link
Contributor Author

@whit537 @kaguillera the latest commit has PayPal and PNC balances matching the statements. Please review and advise on those TODOs and TBDs. Thanks.

@kaguillera
Copy link
Contributor

@whit537 does Braintree take some time to deposit funds after payday the way Stripe did back in the days. If so should we have an Assets:Escrow:Braintree account to show that the funds are being held there before actually being deposited in the Assets:Escrow:New-Alliance account?

The reason I am asking is because that is the way that did it in 2012-07 and I just to make sure that we are be consistent.

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

2012-07-06 * "We Charged participants via Stripe"
    Assets:Escrow:Stripe                  341.61 USD
    Assets:Operations:Stripe               33.63 USD
    Income:Fees:Stripe                    -33.63 USD
    Liabilites:Escrow:Stripe             -341.61 USD

2012-07-19 * "Stripe settlement the net to us"
    Assets:Escrow:New Alliance            341.61 USD
    Assets:Operations:New Alliance          3.52 USD
    Expenses:Fees:Stripe                   30.11 USD
    Assets:Operations:Stripe              -30.11 USD      
    Assets:Operations:Stripe               -3.52 USD
    Assets:Escrow:Stripe                 -341.61 USD

@kaguillera
Copy link
Contributor

Oh @nobodxbodon did you get access to paypal transaction history, cause I see that you need it.

@nobodxbodon
Copy link
Contributor Author

nobodxbodon commented Feb 23, 2017

if so should we have an Assets:Escrow:Braintree account to show that the funds are being held there before actually being deposited in the Assets:Escrow:New-Alliance account?

I suppose that needs Braintree statements? I agree it should be included to reflect the processing fee for the giving part.

@kaguillera I do have paypal transaction reading permission, which I used to get the mass payment numbers, single general payments (1/11, 1/13), and cross-check with new-alliance statements.

@kaguillera kaguillera added Review and removed Review labels Mar 2, 2017
@kaguillera
Copy link
Contributor

Sorry about the label I though it was another PR

Copy link
Contributor

@kaguillera kaguillera left a comment

Choose a reason for hiding this comment

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

Just reviewed after so long. Minor comments let me know what you think @nobodxbodon.


2017-01-09 * "Braintree funding"
Assets:Escrow:New-Alliance 135.54 USD
Liabilities:Escrow -135.54 USD
Copy link
Contributor

Choose a reason for hiding this comment

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

I am assuming this is as a result of running payday. If so I think that you should not it possibly include the payday number. Actually for all the paydays.

@@ -4,204 +4,204 @@
2017-01-01 open Income:Payout USD

Copy link
Contributor

Choose a reason for hiding this comment

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

We could put these in FY2017.beancount since we are using this for the entire year and it is accounts that well will have used befor this fiscal year?

freshdesk: "https://gratipay.freshdesk.com/helpdesk/tickets/6536"
statementPnc: "https://github.com/gratipay/logs/blob/master/statements/2017/01/pnc.csv#L14"
note: "http://gratipay.slackarchive.io/gratipay/-/1483640673.001811/1483653370.001921/1483650056001917/"
Expenses:Operations 34.15 USD
Assets:Operations:PNC -34.15 USD

2017-01-05 * "Payout"
2017-01-05 * "Payout"
freshdesk: "https://gratipay.freshdesk.com/helpdesk/tickets/6530"
statementPnc: "https://github.com/gratipay/logs/blob/master/statements/2017/01/pnc.csv#L13"
Income:Payout -93.57 USD
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a little confusing...I thought that it was mass payout which would have been coming out of Liabilities. Maybe the account should have a different name with reference to Paypal. possibly Income:Funding:Paypal

Liabilities:Escrow 1304.39 USD
Expenses:Fees:PayPal 13.36 USD
Assets:Operations:PayPal -13.36 USD
Assets:Escrow:PayPal -1304.39 USD

2017-01-12 * "Payout"
2017-01-12 * "Payout"
freshdesk: "https://gratipay.freshdesk.com/helpdesk/tickets/6592"
statementPnc: "https://github.com/gratipay/logs/blob/master/statements/2017/01/pnc.csv#L9"
Income:Payout -69.13 USD
Assets:Operations:PNC 69.13 USD

Copy link
Contributor

Choose a reason for hiding this comment

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

Assets:Escrow:New-Alliance 145.30 USD
Liabilities:Escrow -145.30 USD

2017-01-26 * "Payout"
2017-01-26 * "Payout"
freshdesk: "https://gratipay.freshdesk.com/helpdesk/tickets/6691"
statementPnc: "https://github.com/gratipay/logs/blob/master/statements/2017/01/pnc.csv#L5"
Income:Payout -50.70 USD
Assets:Operations:PNC 50.70 USD

Copy link
Contributor

Choose a reason for hiding this comment

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

@kaguillera
Copy link
Contributor

@whit537 let me know what you think.

@kaguillera
Copy link
Contributor

Ok so I have not heard from @nobodxbodon for a while so I am going to finish working on this and let @whit537 review when ever he gets a chance.

@chadwhitacre
Copy link
Contributor

Awesome. It is sitting in my octobox so it is on my radar, fwiw.

@kaguillera
Copy link
Contributor

Ok so I did this one over but I think that I still have a ways to go. I have the following to do before I think it is ready for a proper review. I still would like you @whit537 to look at it and tell me what you think.

  • Check and verify all of the transactions involving PayPal when I get the CSV
  • Figure out how to record Masspay out to show fee paid is part of the escrow reduced.

@kaguillera
Copy link
Contributor

kaguillera commented Apr 21, 2017

The following is how I am considering account for the fact that the fees paid for MassPay come from the recipients. Or should we not bother to record the fee since we don't pay that.

    Liabilities:Escrow               907.58 USD
    Assets:PayPal:Operations          15.11 USD
    Assets:PayPal:Escrow             -15.11 USD
    Assets:PayPal:Escrow            -892.47 USD

    Expenses:Other:Fees               15.11 USD
    Assets:PayPal:Operations         -15.11 USD

Or should I do the following

    Liabilities:Escrow               907.58 USD
    Expenses:Other:Fees               15.11 USD
    Assets:PayPal:Escrow             -15.11 USD
    Assets:PayPal:Escrow            -892.47 USD

@kaguillera
Copy link
Contributor

Yet another approach...

    Liabilities:Escrow               907.58 USD
    Income:PayPal:Fees                15.11 USD
    Assets:PayPal:Escrow             -15.11 USD
    Assets:PayPal:Escrow            -892.47 USD

    Expenses:Other:Fees               15.11 USD
    Imcome:PayPal:Fees               -15.11 USD

@kaguillera
Copy link
Contributor

No success in recording payouts properly. Need help/suggestions.

@chadwhitacre
Copy link
Contributor

chadwhitacre commented Apr 26, 2017

Starting to look at this, a few things jump out at me:

  • Would it be good to consult our CPA on the chart of accounts to use for operating expenses? So far we are introducing a chart of accounts based on File taxes inside.gratipay.com#1027 (comment), but it seems like this is an important decision that we should maybe validate. Peering into the void is giving me vertigo.
  • We're using Beancounts metadata format to record URLs, yes?
  • Can we clean up the metadata keys we're using? Reading up on reconciliation and source documents, maybe we want a single source field (does beancount allow multiple of the same metadata key?)? Ideally we could do something that would support automatic reconciliation.

Now to look at how we're recording MassPay transactions ...

@chadwhitacre
Copy link
Contributor

Cut/paste from @kaguillera at #43:

  • Remember to record money movement from Paypal Gratipay team account to PNC account.
  • How and should we record the movement of money to Assets:Paypal:Escrow/Operations account? but first how do we get money into that account and when…Answer we transfer it from New Alliance during payday shuffle I think. Thus we need to record that in the accounting. I should be able to do this when I get the Paypal csv’s.
  • Do we need to show the expenses that the Teams pay? And if so how do we show it?
  • Need to check all of the transactions involving PayPal when I get the CSV
  • Not sure what to name the Assets:Paypal:Income account. This is the account that captures the money the Gratipay team receives in the team’s paypal account when payday is run.
  • Need to verify if reversals are part of mass pay totals that we send. i.e. if it is sent and then refunded.

This was referenced Apr 26, 2017
@chadwhitacre
Copy link
Contributor

Closing in favor of #44 (standardizing branch name).

@chadwhitacre chadwhitacre deleted the fy201701 branch April 26, 2017 17:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants