[Bug]: Transactions with the same imported_id are not rejected #635

Closed
opened 2026-02-28 19:12:12 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @kenneho on GitHub (Sep 16, 2023).

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

Hi.

I'm using the addTransactions function to upload transactions, and have found that I'm able to upload multiple transactions with the same imported_id. My understanding of https://actualbudget.org/docs/api/reference/#methods is that uploading a transaction with the same imported_id as an existing transaction should fail.

To illustrate, I've ran getTransactions and piped these to a log file, and grepping for imported_id yields this:

imported_id: '2',
imported_id: '2',

What error did you receive?

No response

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Firefox

Operating System

Linux

Originally created by @kenneho on GitHub (Sep 16, 2023). ### Verified issue does not already exist? - [X] I have searched and found no existing issue ### What happened? Hi. I'm using the `addTransactions` function to upload transactions, and have found that I'm able to upload multiple transactions with the same `imported_id`. My understanding of https://actualbudget.org/docs/api/reference/#methods is that uploading a transaction with the same `imported_id` as an existing transaction should fail. To illustrate, I've ran `getTransactions` and piped these to a log file, and grepping for `imported_id` yields this: ``` imported_id: '2', imported_id: '2', ``` ### What error did you receive? _No response_ ### Where are you hosting Actual? Docker ### What browsers are you seeing the problem on? Firefox ### Operating System Linux
GiteaMirror added the bugAPI labels 2026-02-28 19:12:13 -06:00
Author
Owner

@cpainchaud commented on GitHub (Oct 23, 2023):

are you not supposed to use 'importTransaction' ? Isn't it that function which is checking for TID duplication?

@cpainchaud commented on GitHub (Oct 23, 2023): are you not supposed to use 'importTransaction' ? Isn't it that function which is checking for TID duplication?
Author
Owner

@kenneho commented on GitHub (Oct 26, 2023):

are you not supposed to use 'importTransaction' ? Isn't it that function which is checking for TID duplication?

Yeah, you're absolutely right. Fixed this a little while back but forgot to close the issue. Thanks anyways.

@kenneho commented on GitHub (Oct 26, 2023): > are you not supposed to use 'importTransaction' ? Isn't it that function which is checking for TID duplication? Yeah, you're absolutely right. Fixed this a little while back but forgot to close the issue. Thanks anyways.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#635