From 88b452b889a93d86c716030041db20afec77e9ad Mon Sep 17 00:00:00 2001
From: Rick Barrette <rickbarrette@gmail.com>
Date: Sun, 8 Mar 2026 15:04:31 -0400
Subject: [PATCH] use set ID like we do with the other entities

---
 app/models/item.rb                | 5 +++--
 app/services/item_sync_service.rb | 2 +-
 db/migrate/002_create_items.rb    | 1 -
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/item.rb b/app/models/item.rb
index 419ccf9..2aee62e 100644
--- a/app/models/item.rb
+++ b/app/models/item.rb
@@ -11,9 +11,10 @@
 class Item < ApplicationRecord
   belongs_to :issue
 
-  validates :description, presence: true
+  validates_presence_of :id, :description
   validates :unit_price, numericality: { greater_than_or_equal_to: 0 }
-  
+  self.primary_key = :id
+
   # Sync all employees, typically triggered by a scheduled task or manual sync request
   def self.sync
     ItemSyncJob.perform_later(full_sync: true)
diff --git a/app/services/item_sync_service.rb b/app/services/item_sync_service.rb
index 7c1d2bf..9947530 100644
--- a/app/services/item_sync_service.rb
+++ b/app/services/item_sync_service.rb
@@ -24,7 +24,7 @@ class ItemSyncService < SyncServiceBase
 
   # Map relevant attributes from the QBO Employee to the local Employee model
   def process_attributes(local, remote)
-    local.qbo_id  = remote.id
+    local.id  = remote.id
     local.description = remote.description
     local.unit_price = remote.unit_price
   end
diff --git a/db/migrate/002_create_items.rb b/db/migrate/002_create_items.rb
index 378c53f..e90138c 100644
--- a/db/migrate/002_create_items.rb
+++ b/db/migrate/002_create_items.rb
@@ -11,7 +11,6 @@
 class CreateItems < ActiveRecord::Migration[7.0]
   def change
     create_table :items do |t|
-      t.integer :qbo_id, null: false
       t.text    :description, null: false
       t.decimal :unit_price,
                 precision: 15,