Allow specifying of ynab_account_id when calling webhooks

Closes #101
syncforynab · Aug 19, 2018 · 4711066 · 4711066
1 parent e751242
commit 4711066
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/controllers/monzo_controller.rb b/app/controllers/monzo_controller.rb
@@ -34,6 +34,9 @@ def receive
       description << " (Repayment to #{webhook[:data][:counterparty][:name]})"
     end
 
+    # @todo remove the final fall back at some point. It will be a breaking change.
+    ynab_account_id = params[:ynab_account_id] || ENV['YNAB_MONZO_ACCOUNT_ID'] || ENV['YNAB_ACCOUNT_ID']
+
     ynab_creator = YNAB::TransactionCreator.new(
       id: "M#{webhook[:data][:id]}",
       date: Time.parse(webhook[:data][:created]).to_date,
@@ -42,7 +45,7 @@ def receive
       description: description.strip,
       cleared: !foreign_transaction,
       flag: flag,
-      account_id: ENV['YNAB_MONZO_ACCOUNT_ID'] || ENV['YNAB_ACCOUNT_ID'] # @todo remove the fall back at some point. It will be a breaking change
+      account_id: ynab_account_id
     )
 
     create = ynab_creator.create

diff --git a/app/controllers/starling_controller.rb b/app/controllers/starling_controller.rb
@@ -48,7 +48,7 @@ def receive
       description: description.strip,
       cleared: !foreign_transaction,
       flag: flag,
-      account_id: ENV['YNAB_STARLING_ACCOUNT_ID']
+      account_id: params[:ynab_account_id] || ENV['YNAB_STARLING_ACCOUNT_ID']
     )
 
     create = ynab_creator.create