Skip to content
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

Adding transactions merely by date creates double entries #2

Open
tobiasvielmetter opened this issue Oct 6, 2021 · 0 comments
Open

Comments

@tobiasvielmetter
Copy link

tobiasvielmetter commented Oct 6, 2021

Hi, thanks for this great plugin, it helps me streamlining my bookkeeping process.
Yet it seems like transactions are added checking the date only (basically adding all transactions that have been added after the date of the last transaction). This technically seems like a solid approach, but it looks like the date changes on transactions every now and then by a day or two.

My guess is that this issue happens with all payments that take a couple of days to resolve, such as SEPA payments. It does mostly happen with SEPA payments for me.

I am assuming that comparing the reference of a transaction endToEndReference = value["source"] with existing transactions references would circumvent this issue. Looking at the script I was not able to create a workable PR as I am not familiar with the syntax etc, but wanted to point out the issue nevertheless.

Possibly it would be enough to check for if value["status"] == "available" before adding a transaction (status can be either available or pending, assuming the date can still change if the object lists as pending https://stripe.com/docs/api/balance_transactions/list).

See screenshot for an rather extreme example of double entries. Often it's just two entries (payment and fee).

Screenshot 2021-10-06 at 15 07 24

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

No branches or pull requests

1 participant