Add presence validations for required fields (#545)

* Adds basic validations for required fields

Also deletes a few extraneous .keep files

Does not add the family_id required field for user, since that breaks the basic test setup

* Restore keep files to this branch

* Remove Credit model and validate models behind ids

* Restore concerns .keep

app/models/account.rb

@@ -1,6 +1,8 @@
 class Account < ApplicationRecord
   include Syncable
 
+  validates :family, presence: true
+
   broadcasts_refreshes
   belongs_to :family
   has_many :balances, class_name: "AccountBalance"

app/models/account_balance.rb

@@ -1,6 +1,8 @@
 class AccountBalance < ApplicationRecord
   belongs_to :account
 
+  validates :account, :date, :balance, presence: true
+
   scope :in_period, ->(period) { period.date_range.nil? ? all : where(date: period.date_range) }
 
   def self.to_series(account, period = Period.all)

app/models/exchange_rate.rb

@@ -1,4 +1,6 @@
 class ExchangeRate < ApplicationRecord
+  validates :base_currency, :converted_currency, presence: true
+
   def self.convert(from, to, amount)
     return amount unless EXCHANGE_RATE_ENABLED
 

app/models/transaction.rb

@@ -2,6 +2,8 @@ class Transaction < ApplicationRecord
   belongs_to :account
   belongs_to :category, optional: true
 
+  validates :name, :date, :amount, :account, presence: true
+
   after_commit :sync_account
 
   def self.ransackable_attributes(auth_object = nil)

app/models/transaction/category.rb

@@ -2,6 +2,8 @@ class Transaction::Category < ApplicationRecord
   has_many :transactions
   belongs_to :family
 
+  validates :name, :color, :family, presence: true
+
   before_update :clear_internal_category, if: :name_changed?
 
   DEFAULT_CATEGORIES = [

app/models/valuation.rb

@@ -1,6 +1,8 @@
 class Valuation < ApplicationRecord
   belongs_to :account
 
+  validates :account, :date, :value, presence: true
+
   after_commit :sync_account
 
   scope :in_period, ->(period) { period.date_range.nil? ? all : where(date: period.date_range) }