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.

Sign up for GitHub

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

Jump to bottom

Add a test for importing a large number of transactions via CSV #1078

Closed
SimiHunjan opened this issue Oct 24, 2024 · 2 comments
Closed

Add a test for importing a large number of transactions via CSV #1078

SimiHunjan opened this issue Oct 24, 2024 · 2 comments
Assignees
@yiliev0
Milestone
v0.9

Comments

@SimiHunjan
Copy link
Contributor

Problem

Some CSVs have 100+ transactions in them.

Solution

Add a test if we do not have already to testing importing CSVs with a large number of transactions.

Alternatives

No response
@SimiHunjan SimiHunjan changed the title Add a test (if not already) for importing a large number of transactions via CSV Add a test for importing a large number of transactions via CSV Oct 24, 2024
@jbair06 jbair06 added this to the v0.9 milestone Jan 28, 2025
@jbair06
Copy link
Member

jbair06 commented Jan 29, 2025

A test exists, but 100+ transactions is currently too large. It causes the app to freeze, essentially.

@jbair06
Copy link
Member

jbair06 commented Feb 12, 2025

Tests have been added. #1543 has been created for the work needed to resolve the performance issues related to this.

@jbair06 jbair06 closed this as completed Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yiliev0 yiliev0

Labels
None yet
Projects
None yet
Milestone
v0.9
Development

No branches or pull requests
3 participants
@SimiHunjan @jbair06 @yiliev0