Skip to content

Commit ecae3fe

Browse files
authored
Merge pull request #12 from steveredden/remove-family-filter
conditionally filter on family_id
2 parents b639219 + b15ea1f commit ecae3fe

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

app/lib/maybe_client.rb

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,6 @@ def get_families
5656
end
5757

5858
def get_accounts(family_id = nil)
59-
family_id ||= get_families.first&.dig("id")
60-
return [] if family_id.nil?
61-
6259
query = <<-SQL
6360
SELECT
6461
id,
@@ -68,11 +65,15 @@ def get_accounts(family_id = nil)
6865
accountable_type,
6966
subtype
7067
FROM public.accounts
71-
WHERE family_id = $1;
7268
SQL
7369

74-
execute(query, [family_id])
75-
end
70+
if family_id
71+
query += " WHERE family_id = $1"
72+
execute(query, [family_id])
73+
else
74+
execute(query)
75+
end
76+
end
7677

7778
def get_simplefin_transactions(account_id, start_date)
7879
query = <<-SQL

0 commit comments

Comments
 (0)