Add post-sync UI stream updates (#1482)

* Add post-sync UI stream updates

* Fix stream channel id

.gitignore

@@ -6,6 +6,7 @@
 
 # Ignore bundler config.
 /.bundle
+/vendor/bundle
 
 # Ignore all environment files (except templates).
 /.env*

Procfile.dev

@@ -1,3 +1,3 @@
-web: ${DEBUG:+rdbg -O -n -c --} bin/rails server -b 0.0.0.0
-css: bin/rails tailwindcss:watch
+web: bundle exec ${DEBUG:+rdbg -O -n -c --} bin/rails server -b 0.0.0.0
+css: bundle exec bin/rails tailwindcss:watch
 worker: bundle exec good_job start

app/models/account.rb

@@ -99,6 +99,10 @@ class Account < ApplicationRecord
     Holding::Syncer.new(self, start_date: start_date).run
   end
 
+  def post_sync
+    accountable.post_sync
+  end
+
   def original_balance
     balance_amount = balances.chronological.first&.balance || balance
     Money.new(balance_amount, currency)

app/models/account/balance/syncer.rb

@@ -46,6 +46,6 @@ class Account::Balance::Syncer
     end
 
     def is_partial_sync?
-      sync_start_date == provided_start_date && sync_start_date < Date.current
+      sync_start_date == provided_start_date
     end
 end

app/models/concerns/accountable.rb

@@ -33,4 +33,15 @@ module Accountable
   rescue Money::ConversionError
     TimeSeries.new([])
   end
+
+  def post_sync
+    broadcast_remove_to(account, target: "syncing-notification")
+
+    broadcast_replace_to(
+      account,
+      target: "chart_account_#{account.id}",
+      partial: "accounts/show/chart",
+      locals: { account: account }
+    )
+  end
 end

app/models/investment.rb

@@ -54,4 +54,15 @@ class Investment < ApplicationRecord
   def icon
     "line-chart"
   end
+
+  def post_sync
+    broadcast_remove_to(account, target: "syncing-notification")
+
+    broadcast_replace_to(
+      account,
+      target: "chart_account_#{account.id}",
+      partial: account.plaid_account_id.present? ? "investments/chart" : "accounts/show/chart",
+      locals: { account: account }
+    )
+  end
 end

app/views/accounts/show/_chart.html.erb

@@ -1,11 +1,11 @@
-<%# locals: (account:, title: nil, tooltip: nil) %>
+<%# locals: (account:, title: nil, tooltip: nil, **args) %>
 
 <% period = Period.from_param(params[:period]) %>
 <% series = account.series(period: period) %>
 <% trend = series.trend %>
 <% default_value_title = account.asset? ? t(".balance") : t(".owed") %>
 
-<div class="bg-white shadow-xs rounded-xl border border-alpha-black-25 rounded-lg">
+<div id="<%= dom_id(account, :chart) %>" class="bg-white shadow-xs rounded-xl border border-alpha-black-25 rounded-lg">
   <div class="p-4 flex justify-between">
     <div class="space-y-2">
       <div class="flex items-center gap-1">

app/views/accounts/show/_header.html.erb

@@ -21,7 +21,7 @@
 
     <div class="flex items-center gap-3 ml-auto">
       <% unless account.plaid_account_id.present? %>
-        <%= button_to sync_account_path(account), disabled: account.syncing?, class: "flex items-center gap-2", title: "Sync Account" do %>
+        <%= button_to sync_account_path(account), disabled: account.syncing?, data: { turbo: false }, class: "flex items-center gap-2", title: "Sync Account" do %>
           <%= lucide_icon "refresh-cw", class: "w-4 h-4 text-gray-500 hover:text-gray-400" %>
         <% end %>
       <% end %>

app/views/investments/_chart.html.erb

@@ -1,10 +1,10 @@
-<%# locals: (account:) %>
+<%# locals: (account:, **args) %>
 
 <% period = Period.from_param(params[:period]) %>
 <% series = account.series(period: period) %>
 <% trend = series.trend %>
 
-<div class="bg-white shadow-xs rounded-xl border border-alpha-black-25 rounded-lg">
+<div id="<%= dom_id(account, :chart) %>" class="bg-white shadow-xs rounded-xl border border-alpha-black-25 rounded-lg">
   <div class="p-4 flex justify-between">
     <div class="space-y-2">
       <div class="flex items-center gap-1">