feat(payment_terms): Apply nullability for net_payment_term

Author: D1353L

app/models/billing_entity.rb

@@ -96,7 +96,7 @@ class BillingEntity < ApplicationRecord
   validates :document_number_prefix, length: {minimum: 1, maximum: 10}, allow_nil: true, on: :create
   validates :document_number_prefix, length: {minimum: 1, maximum: 10}, on: :update
   validates :invoice_grace_period, numericality: {greater_than_or_equal_to: 0}
-  validates :net_payment_term, numericality: {greater_than_or_equal_to: 0}
+  validates :net_payment_term, numericality: {greater_than_or_equal_to: 0}, allow_nil: true
   validates :logo,
     image: {authorized_content_type: %w[image/png image/jpg image/jpeg], max_size: 800.kilobytes},
     if: :logo?
@@ -209,7 +209,7 @@ def validate_einvoicing
 #  legal_number                                 :string
 #  logo                                         :string
 #  name                                         :string           not null
-#  net_payment_term                             :integer          default(0), not null
+#  net_payment_term                             :integer          default(0)
 #  payment_term                                 :jsonb
 #  phone                                        :string
 #  state                                        :string

app/models/invoice.rb

@@ -773,7 +773,7 @@ def set_finalized_at
 #  finalized_at                            :datetime
 #  invoice_type                            :integer          default("subscription"), not null
 #  issuing_date                            :date
-#  net_payment_term                        :integer          default(0), not null
+#  net_payment_term                        :integer          default(0)
 #  number                                  :string           default(""), not null
 #  payment_attempts                        :integer          default(0), not null
 #  payment_dispute_lost_at                 :datetime

db/migrate/20260819134550_relax_net_payment_term_nullability.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+class RelaxNetPaymentTermNullability < ActiveRecord::Migration[8.0]
+  def change
+    change_column_null :billing_entities, :net_payment_term, true
+    change_column_null :invoices, :net_payment_term, true
+  end
+end

db/structure.sql

@@ -2282,7 +2282,7 @@ CREATE TABLE public.billing_entities (
     finalize_zero_amount_invoice boolean DEFAULT true NOT NULL,
     invoice_footer text,
     invoice_grace_period integer DEFAULT 0 NOT NULL,
-    net_payment_term integer DEFAULT 0 NOT NULL,
+    net_payment_term integer DEFAULT 0,
     email character varying,
     email_settings character varying[] DEFAULT '{}'::character varying[] NOT NULL,
     eu_tax_management boolean DEFAULT false,
@@ -3622,7 +3622,7 @@ CREATE TABLE public.invoices (
     sub_total_excluding_taxes_amount_cents bigint DEFAULT 0 NOT NULL,
     sub_total_including_taxes_amount_cents bigint DEFAULT 0 NOT NULL,
     payment_due_date date,
-    net_payment_term integer DEFAULT 0 NOT NULL,
+    net_payment_term integer DEFAULT 0,
     voided_at timestamp(6) without time zone,
     organization_sequential_id integer DEFAULT 0 NOT NULL,
     ready_to_be_refreshed boolean DEFAULT false NOT NULL,
@@ -14205,6 +14205,7 @@ ALTER TABLE ONLY public.membership_roles
 SET search_path TO "$user", public;
 
 INSERT INTO "schema_migrations" (version) VALUES
+('20260819134550'),
 ('20260819134021'),
 ('20260819111435'),
 ('20260819111434'),

spec/models/billing_entity_spec.rb

@@ -86,6 +86,8 @@
       expect(billing_entity).to be_valid
     end
 
+    it { is_expected.to allow_value(nil).for(:net_payment_term) }
+
     it { is_expected.to validate_length_of(:document_number_prefix).is_at_least(1).is_at_most(10).on(:update) }
 
     it { is_expected.to allow_value(nil).for(:document_number_prefix).on(:create) }