Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

settle the whole graph of tips, not just the first part #2664

Closed
chadwhitacre opened this issue Aug 20, 2014 · 6 comments · Fixed by #2914
Closed

settle the whole graph of tips, not just the first part #2664

chadwhitacre opened this issue Aug 20, 2014 · 6 comments · Fixed by #2914

Comments

@chadwhitacre
Copy link
Contributor

Currently, payday settles the tip graph to a depth of 2. We should settle the whole graph. IRC


Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

@chadwhitacre
Copy link
Contributor Author

Though we also need to take takes into account. We want to settle the whole graph of tips and takes together.

Previous work on this was done in #449 and #1493.

@chadwhitacre
Copy link
Contributor Author

To illustrate, let's say that we have this tip graph:

  • alice tips $100 to bob
  • bob tips $10 to carl
  • carl tips $1 to dana
  • dana tips 10¢ to earl

Only alice has a credit card attached. The way we run now, bob gets his tip, and we let him tip carl, but then we stop. We should let bob tip carl in turn, and carl dana, and dana earl.

@chadwhitacre
Copy link
Contributor Author

Would this be the place to stop carrying balances over from week to week? @Changaco and I discussed that on #2579; @nathany brought it up on #1486.

@chadwhitacre
Copy link
Contributor Author

!m @rohitpaulk @Changaco

@chadwhitacre
Copy link
Contributor Author

Would this be the place to stop carrying balances over from week to week?

Does #2914 address this?

@Changaco
Copy link
Contributor

Changaco commented Dec 4, 2014

@whit537 No.

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

Successfully merging a pull request may close this issue.

2 participants