Docker/Maybe
Docker/Maybe
1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-07-28 17:49:38 +02:00
Code Issues Releases Activity Actions

Fix bug where transactions were duplicated in import confirm (#941)

Browse source 
* Fix bug where transactions were duplicated in import confirm

Signed-off-by: Luke Hurst <hurstlj@umich.edu>

* Update imports_test.rb

Signed-off-by: Luke Hurst <hurstlj@umich.edu>

---------

Signed-off-by: Luke Hurst <hurstlj@umich.edu>
This commit is contained in:
Luke Hurst 2024-07-03 14:44:35 -07:00 • committed by GitHub
parent f1d0a62ac7
commit cf23453d93
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
app/views/imports/confirm.html.erb
View file

@ -13,7 +13,7 @@
<% transaction_entries.group_by(&:date).each do |date, transactions| %> <% transaction_entries.group_by(&:date).each do |date, transactions| %>
<%= render "account/entries/entry_group", <%= render "account/entries/entry_group",
date: date, date: date,
entries: transaction_entries, entries: transactions,
show_tags: true, show_tags: true,
selectable: false, selectable: false,
editable: false %> editable: false %>

1
test/system/imports_test.rb
View file

@ -89,6 +89,7 @@ class ImportsTest < ApplicationSystemTestCase
# 5) Confirm step # 5) Confirm step
assert_selector "h1", text: "Confirm import" assert_selector "h1", text: "Confirm import"
assert_selector "#new_account_entry", count: 2
click_button "Import 2 transactions" click_button "Import 2 transactions"
assert_selector "h1", text: "Imports" assert_selector "h1", text: "Imports"
end end