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

[Bug]: Gocardless import adds declined transactions #1812

Closed
1 task done
benjylxwang opened this issue Oct 17, 2023 · 5 comments
Closed
1 task done

[Bug]: Gocardless import adds declined transactions #1812

benjylxwang opened this issue Oct 17, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@benjylxwang
Copy link

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

If there are declined bank transactions within the bank import window through GoCardless (I'm using Starling UK), they get added to the list of transactions against the linked account. This happens every time you run the import/sync, so every time I sync the account I need to delete the declined transactions or my budget account balance is out of sync with the bank account balance

What error did you receive?

No response

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Firefox, Chrome

Operating System

Windows 11

@benjylxwang benjylxwang added the bug Something isn't working label Oct 17, 2023
@benjylxwang
Copy link
Author

Had a quick look through the code and maybe we could just add a filter on the transactions that come back from the downloadGoCardlessTransactions so that it doesn't return any of the declined ones?

That's here in the codebase.

Only thing I'm not sure about is if that information is in the transaction object at this stage - haven't had a chance to test. The other place I could see it working would be in reconcileGoCardlessTransactions in the same sync.ts file.

@kyrias
Copy link
Contributor

kyrias commented Oct 17, 2023

Had a quick look through the code and maybe we could just add a filter on the transactions that come back from the downloadGoCardlessTransactions so that it doesn't return any of the declined ones?

If done then this should be done in a bank integration in the sync server. https://github.com/actualbudget/actual-server/tree/master/src/app-gocardless/banks

But if the transaction was declined then it should not be served at all and so if anything you should report this as an issue to GoCardless.

@benjylxwang
Copy link
Author

Had a quick look through the code and maybe we could just add a filter on the transactions that come back from the downloadGoCardlessTransactions so that it doesn't return any of the declined ones?

If done then this should be done in a bank integration in the sync server. https://github.com/actualbudget/actual-server/tree/master/src/app-gocardless/banks

But if the transaction was declined then it should not be served at all and so if anything you should report this as an issue to GoCardless.

Ah ok thanks for the clarification on that. I'll report it to GoCardless.

@ACWalker
Copy link
Contributor

ACWalker commented Jul 24, 2024

I've just come across this issue myself with Starling too. I have raised a support issue to GoCardless so will see what they say. Otherwise I will make a bank integration for Starling as @kyrias mentioned.

@ACWalker
Copy link
Contributor

I got a response from gocardless, they said:

I have found the transactions in the our response.  We will look into it to see what might bet the cause why these are returning as "Pending".  I will get back to you as soon as we have any news on the matter.

Will see what they find

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants