-
Notifications
You must be signed in to change notification settings - Fork 308
Conversation
This doesn't indicate actual data corruption, and it isn't affecting user experience, so I'm going to leave this sit for now (not self-merge). I will hopefully see about a test or two after supper. |
|
So I expect 6 or 7 more entries in Sentry when I get back. :-) |
4b05aa4
to
4a566e2
Compare
Actually, this may go deeper. I'm struggling to get a test together. |
I'm looking at a local backup, finding some patterns ... not all balances are failing the check, for one. |
I've whittled down to a failing case with the fewest exchanges. |
Okay, I think I've got it. |
4a566e2
to
b15766d
Compare
Alright! Ready to go. :-) cc: @kaguillera @rohitpaulk @aandis @nobodxbodon @JessaWitzel et al. |
LGTM...I have no complaints |
Looks good to me. |
Updating branch (with GitHub) and waiting for tests. I remember us talking about rebasing instead but I'm not setup for that right now |
ok I merged and deleted the branch. |
https://sentry.io/gratipay/gratipay-com/issues/185206514/
gratipay/inside.gratipay.com#900 (comment)