Fix bug where transactions were duplicated in import confirm

[ljhurst]

What

When reviewing imported transactions on the confirm page, transactions were duplicated for every date in the CSV. This was a view issue only. Clicking import imported the correct number of transactions

http://0.0.0.0:3000/imports/edc55bb1-0fcb-460d-b26b-dfb263728e55/confirm

Example CSV data

date,name,category,amount
2024-07-01,McDonalds,Food & Drink, -10
2024-06-30,Delta,Travel, -100
2024-06-30,Lyft,Auto & Transport, -5

Before

How

• Use grouped transactions instead of all transaction_entries

Validation

After

[zachgoll]

Nice catch and thanks for the fix!

Would you mind also adding this line into the import test so we can make sure this one doesn't come back?

diff --git a/test/system/imports_test.rb b/test/system/imports_test.rb
index b9ee4f8..98f9614 100644
--- a/test/system/imports_test.rb
+++ b/test/system/imports_test.rb
@@ -89,6 +89,7 @@ class ImportsTest < ApplicationSystemTestCase
 
     # 5) Confirm step
     assert_selector "h1", text: "Confirm import"
+    assert_selector "#new_account_entry", count: 2
     click_button "Import 2 transactions"
     assert_selector "h1", text: "Imports"
   end

[ljhurst]

Added!