Fix mode bug

app/models/concerns/accountable.rb

@@ -33,4 +33,8 @@ module Accountable
   rescue Money::ConversionError
     TimeSeries.new([])
   end
+
+  def mode_required?
+    true
+  end
 end

app/models/other_asset.rb

@@ -4,4 +4,8 @@ class OtherAsset < ApplicationRecord
   def color
     "#12B76A"
   end
+
+  def mode_required?
+    false
+  end
 end

app/models/other_liability.rb

@@ -4,4 +4,8 @@ class OtherLiability < ApplicationRecord
   def color
     "#737373"
   end
+
+  def mode_required?
+    false
+  end
 end

app/models/property.rb

@@ -23,6 +23,10 @@ class Property < ApplicationRecord
     "#06AED4"
   end
 
+  def mode_required?
+    false
+  end
+
   private
     def first_valuation_amount
       account.entries.account_valuations.order(:date).first&.amount_money || account.balance_money

app/models/vehicle.rb

@@ -19,6 +19,10 @@ class Vehicle < ApplicationRecord
     "#F23E94"
   end
 
+  def mode_required?
+    false
+  end
+
   private
     def first_valuation_amount
       account.entries.account_valuations.order(:date).first&.amount_money || account.balance_money

app/views/accounts/_form.html.erb

@@ -3,7 +3,9 @@
 <%= styled_form_with model: account, url: url, scope: :account, class: "flex flex-col gap-4 justify-between grow", data: { turbo: false } do |f| %>
   <div class="grow space-y-2">
     <% unless account.new_record? %>
-      <%= f.select :mode, Account::VALUE_MODES.map { |mode| [mode.titleize, mode] }, { label: t(".mode"), prompt: t(".mode_prompt") } %>
+      <% if account.accountable.mode_required? %>
+        <%= f.select :mode, Account::VALUE_MODES.map { |mode| [mode.titleize, mode] }, { label: t(".mode"), prompt: t(".mode_prompt") }, required: true %>
+      <% end %>
     <% end %>
 
     <%= f.select :accountable_type, Accountable::TYPES.map { |type| [type.titleize, type] }, { label: t(".accountable_type"), prompt: t(".type_prompt") }, required: true, autofocus: true %>

app/views/accounts/accountables/_value_onboarding.html.erb

@@ -13,4 +13,4 @@
       <span>Transactions</span>
     <% end %>
   </div>
-</div>
+</div>

app/views/accounts/accountables/credit_card/_overview.html.erb

@@ -28,4 +28,4 @@
 
 <div class="flex justify-center py-8">
   <%= link_to "Edit account details", edit_account_path(account), class: "btn btn--ghost", data: { turbo_frame: :modal } %>
-</div>
+</div>

app/views/accounts/accountables/loan/_overview.html.erb

@@ -46,4 +46,4 @@
 
 <div class="flex justify-center py-8">
   <%= link_to "Edit account details", edit_account_path(account), class: "btn btn--ghost", data: { turbo_frame: :modal } %>
-</div>
+</div>