Enable syncing all accounts in one click (#948)

* Enable syncing all accounts on one click

* Remove argument to sync_later method call

* Add partial for sync all accounts button

* Redirect back if possible when syncing all accounts

app/controllers/accounts_controller.rb

@@ -69,6 +69,22 @@ class AccountsController < ApplicationController
     redirect_to account_path(@account), notice: t(".success")
   end
 
+  def sync_all
+    synced_accounts_count = 0
+    Current.family.accounts.each do |account|
+      next unless account.can_sync?
+
+      account.sync_later
+      synced_accounts_count += 1
+    end
+
+    if synced_accounts_count > 0
+      redirect_back_or_to accounts_path, notice: t(".success", count: synced_accounts_count)
+    else
+      redirect_back_or_to accounts_path, alert: t(".no_accounts_to_sync")
+    end
+  end
+
   private
 
     def set_account

app/views/accounts/_header.html.erb

@@ -17,5 +17,7 @@
       <%= lucide_icon("plus", class: "w-5 h-5") %>
       <p class="text-sm font-medium"><%= t(".new") %></p>
     <% end %>
+
+    <%= render "sync_all_button" %>
   </div>
 </header>

app/views/accounts/_sync_all_button.html.erb

@@ -0,0 +1,3 @@
+<%= button_to sync_all_accounts_path, method: :post, class: "rounded-lg bg-gray-900 text-white flex items-center gap-1 justify-center hover:bg-gray-700 px-3 py-2", title: "Sync All" do %>
+  <%= lucide_icon "refresh-cw", class: "w-5 h-5" %>
+<% end %>

app/views/accounts/index.html.erb

@@ -24,6 +24,8 @@
           <%= lucide_icon("plus", class: "w-5 h-5") %>
           <p class="text-sm font-medium"><%= t(".new_account") %></p>
         <% end %>
+
+        <%= render "sync_all_button" %>
       </div>
     </div>
   </header>

config/locales/views/accounts/en.yml

@@ -65,5 +65,8 @@ en:
       new: New account
     sync:
       success: Account sync started
+    sync_all:
+      no_accounts_to_sync: No accounts were eligible for syncing.
+      success: Successfully queued %{count} accounts for syncing.
     update:
       success: Account updated

config/routes.rb

@@ -63,6 +63,7 @@ Rails.application.routes.draw do
     collection do
       get :summary
       get :list
+      post :sync_all
     end
 
     member do

test/controllers/accounts_controller_test.rb

@@ -30,6 +30,12 @@ class AccountsControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to account_url(@account)
   end
 
+  test "can sync all accounts" do
+    post sync_all_accounts_path
+    assert_redirected_to accounts_url
+    assert_equal "Successfully queued #{ @user.family.accounts.size } accounts for syncing.", flash[:notice]
+  end
+
   test "should update account" do
     patch account_url(@account), params: {
       account: {